emscripten-core / emscripten-core/emscripten

Enable ABORTING_MALLOC by default when ALLOW_MEMORY_GROWTH is set?

Offen
#19,600 0 Kommentare 2 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
C++
Sterne
27.6k
Forks
3.6k
Ø Merge
1 T. 14 Std.
Gemergte PRs (30 T.)
125

Beschreibung

Currently it looks like ABORTING_MALLOC defaults to
- ON if building with memory growth disabled, and to
- OFF, when building with allow memory growth enabled.

This difference seems arbitrary. Having aborting malloc is orthogonal to whether memory growth is enabled, so it would be best to have aborting malloc always enabled by default to be consistent? Naturally in ALLOW_MEMORY_GROWTH+ABORTING_MALLOC builds, one should only abort after resizing the wasm heap fails. (like it already does happen)

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginne damit, die standardmäßige Behandlung von ABORTING_MALLOC und ALLOW_MEMORY_GROWTH in den Emscripten-Compilerquellen zu lokalisieren, und untersuche anschließend das bestehende Verhalten beim Vergrößern des wasm-Heaps und bei fehlgeschlagenen Allokationen. Als erledigt gilt die Aufgabe, wenn ABORTING_MALLOC unabhängig vom Speicherwachstum standardmäßig aktiviert ist, während Builds mit aktiviertem Wachstum den Speicher weiterhin vor dem Abbruch vergrößern, wenn die Vergrößerung fehlschlägt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
cpp, wasm
Bereich
compilers
Issue-Typ
Feature
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.