Mark WASM_BIGINT as deprecated - #27558
Merged
Merged
Conversation
guybedford
approved these changes
Aug 17, 2026
sbc100
force-pushed
the
deprecate_wasm_bigint
branch
4 times, most recently
from
August 17, 2026 23:08
53f4bfa to
aacd30c
Compare
sbc100
force-pushed
the
deprecate_wasm_bigint
branch
2 times, most recently
from
August 18, 2026 00:31
3eac9d9 to
d91112d
Compare
sbc100
changed the title
sbc100
force-pushed
the
deprecate_wasm_bigint
branch
from
August 18, 2026 00:33
d91112d to
379c882
Compare
kripken
reviewed
Aug 18, 2026
kripken
reviewed
Aug 18, 2026
kripken
reviewed
Aug 18, 2026
sbc100
added a commit
to sbc100/emscripten
that referenced
this pull request
Aug 18, 2026
Split out from emscripten-core#27558.
sbc100
added a commit
to sbc100/emscripten
that referenced
this pull request
Aug 18, 2026
Split out from emscripten-core#27558.
sbc100
added a commit
to sbc100/emscripten
that referenced
this pull request
Aug 18, 2026
Split out from emscripten-core#27558.
sbc100
added a commit
that referenced
this pull request
Aug 19, 2026
This setting has already been deprecated for a long time. Flipping its default is really no-op here but it makes the followup changes to remove it simpler. Split out from #27558.
sbc100
added a commit
to sbc100/emscripten
that referenced
this pull request
Aug 19, 2026
Move `LEGALIZE_JS_FFI` to `LEGACY_SETTINGS` (accepting only `0` for backwards compatibility). JS FFI legalization is no longer needed under the default `WASM_BIGINT` configuration and is now conditioned directly on `WASM_BIGINT=0`. Split out from emscripten-core#27558.
sbc100
added a commit
to sbc100/emscripten
that referenced
this pull request
Aug 19, 2026
Move `LEGALIZE_JS_FFI` to `LEGACY_SETTINGS` (accepting only `0` for backwards compatibility). JS FFI legalization is no longer needed under the default `WASM_BIGINT` configuration and is now conditioned directly on `WASM_BIGINT=0`. Split out from emscripten-core#27558.
sbc100
added a commit
that referenced
this pull request
Aug 19, 2026
Move `LEGALIZE_JS_FFI` to `LEGACY_SETTINGS` (accepting only `0` for backwards compatibility). JS FFI legalization is no longer needed under the default `WASM_BIGINT` configuration and is now conditioned directly on `WASM_BIGINT=0`. Split out from #27558.
sbc100
added a commit
to sbc100/emscripten
that referenced
this pull request
Aug 19, 2026
In `wasm-emscripten-finalize`, both `--bigint` and `--no-legalize-javascript-ffi` have the exact same effect: they prevent running the `legalize-js-interface` pass (`if (!bigInt && legalizeJavaScriptFFI)`). Previously, when `WASM_BIGINT` was enabled, Emscripten was passing both `--bigint` and `--no-legalize-javascript-ffi`. In addition, when `AUTODEBUG` was enabled, `--bigint` was appended a second time. Consolidate the check so `--bigint` is passed once when either `WASM_BIGINT` or `AUTODEBUG` is enabled, and remove the redundant `--no-legalize-javascript-ffi` flag. Split out from emscripten-core#27558.
sbc100
added a commit
that referenced
this pull request
Aug 19, 2026
In `wasm-emscripten-finalize`, both `--bigint` and `--no-legalize-javascript-ffi` have the exact same effect: they prevent running the `legalize-js-interface` pass (`if (!bigInt && legalizeJavaScriptFFI)`). Previously, when `WASM_BIGINT` was enabled, Emscripten was passing both `--bigint` and `--no-legalize-javascript-ffi`. In addition, when `AUTODEBUG` was enabled, `--bigint` was appended a second time. Consolidate the check so `--bigint` is passed once when either `WASM_BIGINT` or `AUTODEBUG` is enabled, and remove the redundant `--no-legalize-javascript-ffi` flag. Split out from #27558.
sbc100
changed the title
sbc100
force-pushed
the
deprecate_wasm_bigint
branch
2 times, most recently
from
August 19, 2026 22:43
0aa057c to
45a6db8
Compare
kripken
approved these changes
Aug 19, 2026
With the minimum Safari version bumped to 15 in emscripten-core#27542, WebAssembly BigInt integration (`WASM_BIGINT`) is now standard and enabled by default across all supported engines. The only remaining use case for disabling it is internally for `WASM2JS` (`-sWASM=0`). Users should no longer set `-sWASM_BIGINT` directly, so mark `WASM_BIGINT` as deprecated.
sbc100
force-pushed
the
deprecate_wasm_bigint
branch
from
August 20, 2026 00:22
45a6db8 to
3c8872f
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
With the minimum Safari version bumped to 15 in #27542, WebAssembly
BigInt integration (
WASM_BIGINT) is now standard and enabled bydefault across all supported engines. The only remaining use case
for disabling it is internally for
WASM2JS(-sWASM=0).Users should no longer set
-sWASM_BIGINTdirectly, so markWASM_BIGINTas deprecated.