Skip to content

Move LEGALIZE_JS_FFI to legacy settings list - #27568

Merged
sbc100 merged 1 commit into
emscripten-core:mainfrom
sbc100:deprecate_legalize_js_ffi
Aug 19, 2026
Merged

Move LEGALIZE_JS_FFI to legacy settings list#27568
sbc100 merged 1 commit into
emscripten-core:mainfrom
sbc100:deprecate_legalize_js_ffi

Conversation

@sbc100

@sbc100 sbc100 commented Aug 19, 2026

Copy link
Copy Markdown
Collaborator

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
sbc100 requested a review from kripken August 19, 2026 17:31
@sbc100 sbc100 changed the title Remove LEGALIZE_JS_FFI setting Move LEGALIZE_JS_FFI to legacy settings list Aug 19, 2026
@sbc100 sbc100 changed the title Move LEGALIZE_JS_FFI to legacy settings list Move LEGALIZE_JS_FFI to legacy settings list Aug 19, 2026
@sbc100

sbc100 commented Aug 19, 2026

Copy link
Copy Markdown
Collaborator Author

I'm gonna land #27569 first which makes this change even smaller.

@sbc100
sbc100 force-pushed the deprecate_legalize_js_ffi branch from 2fc5d96 to 9e2405c Compare August 19, 2026 19:37
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
sbc100 force-pushed the deprecate_legalize_js_ffi branch from 9e2405c to 5d9c9da Compare August 19, 2026 19:40
@sbc100
sbc100 merged commit ddff96a into emscripten-core:main Aug 19, 2026
21 of 24 checks passed
@sbc100
sbc100 deleted the deprecate_legalize_js_ffi branch August 19, 2026 19:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants