Skip to content

Commit 6329052

Browse files
sbc100hedwigz
authored andcommitted
[wasm64] Remove table64 lowering by default (emscripten-core#23051)
The engines that support memory64 now all also support table64.
1 parent 9536cc8 commit 6329052

File tree

1 file changed

+4
-5
lines changed

1 file changed

+4
-5
lines changed

tools/link.py

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -2005,11 +2005,10 @@ def run_embind_gen(wasm_target, js_syms, extra_settings, linker_inputs):
20052005
# Build the flags needed by Node.js to properly run the output file.
20062006
node_args = []
20072007
if settings.MEMORY64:
2008-
node_args += shared.node_memory64_flags()
2009-
# Currently we don't have any engines that support table64 so we need
2010-
# to lower it in order to run the output.
2011-
# In the normal flow this happens later in `phase_binaryen`
2012-
building.run_wasm_opt(outfile_wasm, outfile_wasm, ['--table64-lowering'])
2008+
# The final version of memory64 proposal is not yet implement in any
2009+
# shipping version of node, so we need to lower it away in order to
2010+
# execute the binary at built time.
2011+
building.run_wasm_opt(outfile_wasm, outfile_wasm, ['--memory64-lowering', '--table64-lowering'])
20132012
if settings.WASM_EXCEPTIONS:
20142013
node_args += shared.node_exception_flags(config.NODE_JS)
20152014
# Run the generated JS file with the proper flags to generate the TypeScript bindings.

0 commit comments

Comments
 (0)