Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable WASM_BIGINT by default #22993

Merged
merged 46 commits into from
Dec 18, 2024
Merged
Show file tree
Hide file tree
Changes from 45 commits
Commits
Show all changes
46 commits
Select commit Hold shift + click to select a range
9eaa392
[WIP] Enable WASM_BIGINT by default
dschuff Nov 15, 2024
5532f29
review comments; fix name section emission
dschuff Nov 22, 2024
5d13302
one more debuginfo test works now
dschuff Nov 22, 2024
152c205
fix more tests, use explicit need_name_section in finalize
dschuff Nov 23, 2024
4a6588e
Use i53abi for one more wasmfs function
dschuff Nov 27, 2024
52fedfd
update node version for bigint
dschuff Nov 27, 2024
01d5e34
dont use i53 in the wrong place
dschuff Dec 6, 2024
e19febb
Merge branch 'main' into bigint_on
dschuff Dec 6, 2024
2c1b640
rebaseline size tests
dschuff Dec 6, 2024
c0f2c4a
Merge branch 'main' into bigint_on
dschuff Dec 6, 2024
1f0232b
update browser.test_small_js_flags size
dschuff Dec 7, 2024
3cbc37c
Merge branch 'main' into bigint_on
dschuff Dec 10, 2024
b9c8be1
fix WasmFS API
dschuff Dec 11, 2024
fb50f90
Merge branch 'main' into bigint_on
dschuff Dec 11, 2024
513ce14
rebaseline
dschuff Dec 11, 2024
8185db1
fix test expectations
dschuff Dec 13, 2024
22165af
rebaseline
dschuff Dec 13, 2024
dd34e39
set WASM_BIGINT setting, fix tests
dschuff Dec 13, 2024
166dcc7
Merge branch 'main' into bigint_on
dschuff Dec 13, 2024
3088726
fix doc, disable bigint with old node
dschuff Dec 13, 2024
f81e0da
fix old node
dschuff Dec 13, 2024
833c212
don't upgrade browser versions based on bigint after it's the default
dschuff Dec 14, 2024
4004df6
update settings doc
dschuff Dec 14, 2024
faa22bb
Merge branch 'main' into bigint_on
dschuff Dec 14, 2024
c2627fd
fix checking of BIGINT against explicit version
dschuff Dec 14, 2024
382211e
fix TODOs to point to the right bug
dschuff Dec 16, 2024
b05455c
Merge branch 'main' into bigint_on
dschuff Dec 16, 2024
2d5fd55
Merge branch 'main' into bigint_on
dschuff Dec 16, 2024
2cc1703
fix copypasta
dschuff Dec 16, 2024
a8372fc
remove unneeded disabling of BIGINT, update comment, add explicit con…
dschuff Dec 16, 2024
cb4503e
rebaseline
dschuff Dec 16, 2024
ed8906a
review comments, fix error check
dschuff Dec 17, 2024
88775c6
change disabling text
dschuff Dec 17, 2024
dd73366
Merge branch 'main' into bigint_on
dschuff Dec 17, 2024
b70d46a
Merge branch 'main' into bigint_on
dschuff Dec 17, 2024
2ee2ce3
remove debug print
dschuff Dec 18, 2024
759de04
fix sanity.test_binaryen_version
dschuff Dec 18, 2024
f3c20a3
fix test_dyncall_specific
dschuff Dec 18, 2024
4abe5fb
fix test_binaryen, fix min_browser_version
dschuff Dec 18, 2024
b7d960a
add changelog, fix dyncall_specific again
dschuff Dec 18, 2024
8a3df8f
Merge branch 'main' into bigint_on
dschuff Dec 18, 2024
d9fb51c
clarify comment
dschuff Dec 18, 2024
ff8c662
Merge branch 'main' into bigint_on
dschuff Dec 18, 2024
9194564
Merge branch 'main' into bigint_on
dschuff Dec 18, 2024
8e5ba19
rebaseline test_small_js_flags, and test_INCOMING_MODULE_JS_API
dschuff Dec 18, 2024
02bc462
add PR number to changelog
dschuff Dec 18, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions ChangeLog.md
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,9 @@ See docs/process.md for more on how version tagging works.

3.1.75 (in development)
-----------------------
- The `WASM_BIGINT` feature has been enabled by default. This has the effect that
Wasm i64 values are passed and returned between Wasm and JS as BigInt values
rather than being split by Binaryen into pairs of Numbers.
dschuff marked this conversation as resolved.
Show resolved Hide resolved
- When using `-sMODULARIZE` we now assert if the factory function is called with
the JS `new` keyword. e.g. `a = new Module()` rather than `b = Module()`.
This paves the way for marking the function as `async` which does not allow
Expand Down
2 changes: 1 addition & 1 deletion site/source/docs/tools_reference/settings_reference.rst
Original file line number Diff line number Diff line change
Expand Up @@ -2189,7 +2189,7 @@ legalize i64s into pairs of i32s, as the wasm VM will use a BigInt where an
i64 is used. If WASM_BIGINT is present, the default minimum supported browser
versions will be increased to the min version that supports BigInt.

Default value: false
Default value: true

.. _emit_producers_section:

Expand Down
2 changes: 1 addition & 1 deletion src/settings.js
Original file line number Diff line number Diff line change
Expand Up @@ -1498,7 +1498,7 @@ var DYNCALLS = false;
// i64 is used. If WASM_BIGINT is present, the default minimum supported browser
// versions will be increased to the min version that supports BigInt.
// [link]
var WASM_BIGINT = false;
var WASM_BIGINT = true;

// WebAssembly defines a "producers section" which compilers and tools can
// annotate themselves in, and LLVM emits this by default.
Expand Down
3 changes: 2 additions & 1 deletion src/shell.js
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,8 @@ var Module = typeof {{{ EXPORT_NAME }}} != 'undefined' ? {{{ EXPORT_NAME }}} : {
#endif // USE_CLOSURE_COMPILER

#if POLYFILL
#if WASM_BIGINT && MIN_SAFARI_VERSION < 150000
#if WASM_BIGINT && MIN_SAFARI_VERSION < 140100
// TODO(https://github.com/emscripten-core/emscripten/issues/23184): Fix this back to 150000
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't follow these changes. This will make the polyfill be added in fewer cases - why is that correct?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The point of the general messing with safari versions ( in feature_matrix itself) is to turn BIGINT on by default without affecting other features. This one here is just so that we won't default to having BIGINT on with this polyfill.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, thanks, I see. So the goal is to split out the safari version change? If so, I wonder if we can do that in this PR? The complexity of splitting it seems relatively high to me.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well, the real goal is to avoid changing all 3 settings in a single commit. But the commits also tie the settings more directly to the browser versions. So the simplest way to make enabling the features independent from each other (and independent from other consequences of raising the default version target) was to lower the version target for each feature separately (which should be a 2-line change for this feature, and 1-line for the others).

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, sorry, but why was that the real goal? Naively I'd think that adjusting the browser versions would be the natural way to enable features by default, so it fits in this PR, so I'm still missing something, sorry.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hm. I guess after the release we aren't really bisecting like you do with revisions, we're just testing flags to find out what breaks. So the options are testing by enabling or disabling the feature flags, or by setting the fake MIN_SAFARI version numbers and seeing what changes. But at that point the feature flags will still be there, so it's not clear to me what keeping the fake safari versions adds.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome!!!

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hm. I guess after the release we aren't really bisecting like you do with revisions, we're just testing flags to find out what breaks.

Was this in reference to my mention of bisecting on MIN_SAFARI?

If so then my point there was that if we add Safari 14.7, 14.8 etc., avoiding a single version that adds multiple features, then bisecting MIN_SAFARI in the future becomes more useful. I agree that using the feature flags could also be done instead, but that requires one to know which flags are relevant and manually set them up, while bisecting on MIN_SAFARI can be done by just mindless bisection on a range of numbers. Like mindless bisection on commits, that sounds useful.

Anyhow this is a minor advantage of 14.7, 14.8 imo. The larger benefit as it I see it is that once we decide on the 14.7 etc. scheme, it is permanent: we have nothing to revert in a later PR, and we have no odd things that show up during bisection that were temporary before being undone. It is a design that "smooths" out safari versions to make them incremental. If browsers enable multiple features in a single release in the future then the same idea could be used again, that is, I don't see this as a one-off hack: it is a principled approach to this problem, as I see it.

But, again, if no one else sees value in this, that's fine 😄 I'm ok with the alternative, the difference is not huge here.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, I think mainly where we disagree is that I don't really like the idea of keeping fake browser versions around in the codebase. Bisecting on browser versions isn't really mindless if you have to know that the fake version numbers exist and what they are for, and what they can be set to.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fair enough, sgtm. Maybe I like the idea of smoothing the history because it sounds like a math operation... but I agree fake versions have the downside you mention too.

// See https://caniuse.com/mdn-javascript_builtins_bigint64array
#include "polyfill/bigint64array.js"
dschuff marked this conversation as resolved.
Show resolved Hide resolved
#endif
Expand Down
2 changes: 1 addition & 1 deletion test/browser/test_small_js_flags.js.size
Original file line number Diff line number Diff line change
@@ -1 +1 @@
4297
4380
12 changes: 6 additions & 6 deletions test/code_size/embind_hello_wasm.json
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
{
"a.html": 552,
"a.html.gz": 380,
"a.js": 9593,
"a.js.gz": 4230,
"a.wasm": 7615,
"a.wasm.gz": 3471,
"total": 17760,
"total_gz": 8081
"a.js": 9879,
"a.js.gz": 4288,
"a.wasm": 7580,
"a.wasm.gz": 3449,
"total": 18011,
"total_gz": 8117
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sad that these numbers go up and not down :(

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Embind didn't support 64bit types before, so now there's some more JS code to handle bigint types. I'm guessing a real world example would show a very small change or improvement.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, most of the other changes are going down. And we'll have more improvements to come with the other features.

}
12 changes: 6 additions & 6 deletions test/code_size/embind_val_wasm.json
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
{
"a.html": 552,
"a.html.gz": 380,
"a.js": 6724,
"a.js.gz": 2900,
"a.wasm": 9528,
"a.wasm.gz": 4896,
"total": 16804,
"total_gz": 8176
"a.js": 7153,
"a.js.gz": 3042,
"a.wasm": 9493,
"a.wasm.gz": 4872,
"total": 17198,
"total_gz": 8294
}
4 changes: 3 additions & 1 deletion test/common.py
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@
from tools.shared import get_canonical_temp_dir, path_from_root
from tools.utils import MACOS, WINDOWS, read_file, read_binary, write_binary, exit_with_error
from tools.settings import COMPILE_TIME_SETTINGS
from tools import shared, line_endings, building, config, utils
from tools import shared, feature_matrix, line_endings, building, config, utils

logger = logging.getLogger('common')

Expand Down Expand Up @@ -1164,6 +1164,8 @@ def setUp(self):
if node_version < emcc_min_node_version:
self.emcc_args += building.get_emcc_node_flags(node_version)
self.emcc_args.append('-Wno-transpile')
if node_version[0] < feature_matrix.min_browser_versions[feature_matrix.Feature.JS_BIGINT_INTEGRATION]['node'] / 10000:
self.emcc_args.append('-sWASM_BIGINT=0')

self.v8_args = ['--wasm-staging']
self.env = {}
Expand Down
5 changes: 0 additions & 5 deletions test/other/codesize/test_codesize_cxx_ctors1.exports
Original file line number Diff line number Diff line change
Expand Up @@ -2,11 +2,6 @@ __indirect_function_table
__wasm_call_ctors
_emscripten_stack_alloc
_emscripten_stack_restore
dynCall_iiiiiijj
dynCall_iiiiij
dynCall_iiiiijj
dynCall_jiji
dynCall_viijii
emscripten_stack_get_current
main
memory
2 changes: 1 addition & 1 deletion test/other/codesize/test_codesize_cxx_ctors1.gzsize
Original file line number Diff line number Diff line change
@@ -1 +1 @@
8502
8352
2 changes: 1 addition & 1 deletion test/other/codesize/test_codesize_cxx_ctors1.jssize
Original file line number Diff line number Diff line change
@@ -1 +1 @@
20867
20347
2 changes: 1 addition & 1 deletion test/other/codesize/test_codesize_cxx_ctors1.size
Original file line number Diff line number Diff line change
@@ -1 +1 @@
129231
128974
5 changes: 0 additions & 5 deletions test/other/codesize/test_codesize_cxx_ctors2.exports
Original file line number Diff line number Diff line change
@@ -1,11 +1,6 @@
__indirect_function_table
_emscripten_stack_alloc
_emscripten_stack_restore
dynCall_iiiiiijj
dynCall_iiiiij
dynCall_iiiiijj
dynCall_jiji
dynCall_viijii
emscripten_stack_get_current
main
memory
2 changes: 1 addition & 1 deletion test/other/codesize/test_codesize_cxx_ctors2.gzsize
Original file line number Diff line number Diff line change
@@ -1 +1 @@
8486
8336
2 changes: 1 addition & 1 deletion test/other/codesize/test_codesize_cxx_ctors2.jssize
Original file line number Diff line number Diff line change
@@ -1 +1 @@
20835
20315
2 changes: 1 addition & 1 deletion test/other/codesize/test_codesize_cxx_ctors2.size
Original file line number Diff line number Diff line change
@@ -1 +1 @@
128680
128423
6 changes: 0 additions & 6 deletions test/other/codesize/test_codesize_cxx_except.exports
Original file line number Diff line number Diff line change
Expand Up @@ -8,12 +8,6 @@ __wasm_call_ctors
_emscripten_stack_alloc
_emscripten_stack_restore
_emscripten_tempret_set
dynCall_iiiiiijj
dynCall_iiiiij
dynCall_iiiiijj
dynCall_jiiii
dynCall_jiji
dynCall_viijii
emscripten_stack_get_current
main
memory
Expand Down
2 changes: 1 addition & 1 deletion test/other/codesize/test_codesize_cxx_except.gzsize
Original file line number Diff line number Diff line change
@@ -1 +1 @@
9533
9358
2 changes: 1 addition & 1 deletion test/other/codesize/test_codesize_cxx_except.jssize
Original file line number Diff line number Diff line change
@@ -1 +1 @@
24712
24116
2 changes: 1 addition & 1 deletion test/other/codesize/test_codesize_cxx_except.size
Original file line number Diff line number Diff line change
@@ -1 +1 @@
171361
171008
5 changes: 0 additions & 5 deletions test/other/codesize/test_codesize_cxx_except_wasm.exports
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,5 @@ __indirect_function_table
__trap
__wasm_call_ctors
_emscripten_stack_alloc
dynCall_iiiiiijj
dynCall_iiiiij
dynCall_iiiiijj
dynCall_jiji
dynCall_viijii
main
memory
2 changes: 1 addition & 1 deletion test/other/codesize/test_codesize_cxx_except_wasm.gzsize
Original file line number Diff line number Diff line change
@@ -1 +1 @@
8464
8321
2 changes: 1 addition & 1 deletion test/other/codesize/test_codesize_cxx_except_wasm.jssize
Original file line number Diff line number Diff line change
@@ -1 +1 @@
20760
20240
2 changes: 1 addition & 1 deletion test/other/codesize/test_codesize_cxx_except_wasm.size
Original file line number Diff line number Diff line change
@@ -1 +1 @@
142481
142223
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,5 @@ __indirect_function_table
__trap
__wasm_call_ctors
_emscripten_stack_alloc
dynCall_iiiiiijj
dynCall_iiiiij
dynCall_iiiiijj
dynCall_jiji
dynCall_viijii
main
memory
Original file line number Diff line number Diff line change
@@ -1 +1 @@
8464
8321
Original file line number Diff line number Diff line change
@@ -1 +1 @@
20760
20240
Original file line number Diff line number Diff line change
@@ -1 +1 @@
145068
144810
2 changes: 1 addition & 1 deletion test/other/codesize/test_codesize_cxx_lto.gzsize
Original file line number Diff line number Diff line change
@@ -1 +1 @@
8413
8348
16 changes: 8 additions & 8 deletions test/other/codesize/test_codesize_cxx_lto.imports
Original file line number Diff line number Diff line change
@@ -1,11 +1,11 @@
a (fd_seek)
b (emscripten_resize_heap)
c (_tzset_js)
d (_setitimer_js)
e (_emscripten_runtime_keepalive_clear)
f (_abort_js)
g (proc_exit)
h (fd_write)
a (emscripten_resize_heap)
b (_tzset_js)
c (_setitimer_js)
d (_emscripten_runtime_keepalive_clear)
e (_abort_js)
f (proc_exit)
g (fd_write)
h (fd_seek)
i (fd_read)
j (fd_close)
k (environ_sizes_get)
Expand Down
2 changes: 1 addition & 1 deletion test/other/codesize/test_codesize_cxx_lto.jssize
Original file line number Diff line number Diff line change
@@ -1 +1 @@
20451
20371
16 changes: 8 additions & 8 deletions test/other/codesize/test_codesize_cxx_lto.sent
Original file line number Diff line number Diff line change
@@ -1,11 +1,11 @@
a (fd_seek)
b (emscripten_resize_heap)
c (_tzset_js)
d (_setitimer_js)
e (_emscripten_runtime_keepalive_clear)
f (_abort_js)
g (proc_exit)
h (fd_write)
a (emscripten_resize_heap)
b (_tzset_js)
c (_setitimer_js)
d (_emscripten_runtime_keepalive_clear)
e (_abort_js)
f (proc_exit)
g (fd_write)
h (fd_seek)
i (fd_read)
j (fd_close)
k (environ_sizes_get)
Expand Down
2 changes: 1 addition & 1 deletion test/other/codesize/test_codesize_cxx_lto.size
Original file line number Diff line number Diff line change
@@ -1 +1 @@
121923
121924
6 changes: 0 additions & 6 deletions test/other/codesize/test_codesize_cxx_mangle.exports
Original file line number Diff line number Diff line change
Expand Up @@ -9,12 +9,6 @@ __wasm_call_ctors
_emscripten_stack_alloc
_emscripten_stack_restore
_emscripten_tempret_set
dynCall_iiiiiijj
dynCall_iiiiij
dynCall_iiiiijj
dynCall_jiiii
dynCall_jiji
dynCall_viijii
emscripten_stack_get_current
free
main
Expand Down
2 changes: 1 addition & 1 deletion test/other/codesize/test_codesize_cxx_mangle.gzsize
Original file line number Diff line number Diff line change
@@ -1 +1 @@
9538
9361
2 changes: 1 addition & 1 deletion test/other/codesize/test_codesize_cxx_mangle.jssize
Original file line number Diff line number Diff line change
@@ -1 +1 @@
24712
24116
2 changes: 1 addition & 1 deletion test/other/codesize/test_codesize_cxx_mangle.size
Original file line number Diff line number Diff line change
@@ -1 +1 @@
232870
232517
5 changes: 0 additions & 5 deletions test/other/codesize/test_codesize_cxx_noexcept.exports
Original file line number Diff line number Diff line change
Expand Up @@ -2,11 +2,6 @@ __indirect_function_table
__wasm_call_ctors
_emscripten_stack_alloc
_emscripten_stack_restore
dynCall_iiiiiijj
dynCall_iiiiij
dynCall_iiiiijj
dynCall_jiji
dynCall_viijii
emscripten_stack_get_current
main
memory
2 changes: 1 addition & 1 deletion test/other/codesize/test_codesize_cxx_noexcept.gzsize
Original file line number Diff line number Diff line change
@@ -1 +1 @@
8502
8352
2 changes: 1 addition & 1 deletion test/other/codesize/test_codesize_cxx_noexcept.jssize
Original file line number Diff line number Diff line change
@@ -1 +1 @@
20867
20347
2 changes: 1 addition & 1 deletion test/other/codesize/test_codesize_cxx_noexcept.size
Original file line number Diff line number Diff line change
@@ -1 +1 @@
132038
131781
8 changes: 0 additions & 8 deletions test/other/codesize/test_codesize_cxx_wasmfs.exports
Original file line number Diff line number Diff line change
Expand Up @@ -2,14 +2,6 @@ __indirect_function_table
__wasm_call_ctors
_emscripten_stack_alloc
_emscripten_stack_restore
dynCall_iiiiiijj
dynCall_iiiiij
dynCall_iiiiijj
dynCall_iiiij
dynCall_iij
dynCall_ji
dynCall_jiji
dynCall_viijii
emscripten_stack_get_current
main
memory
2 changes: 1 addition & 1 deletion test/other/codesize/test_codesize_cxx_wasmfs.gzsize
Original file line number Diff line number Diff line change
@@ -1 +1 @@
3769
3653
2 changes: 1 addition & 1 deletion test/other/codesize/test_codesize_cxx_wasmfs.jssize
Original file line number Diff line number Diff line change
@@ -1 +1 @@
8493
7914
2 changes: 1 addition & 1 deletion test/other/codesize/test_codesize_cxx_wasmfs.size
Original file line number Diff line number Diff line change
@@ -1 +1 @@
169290
168928
2 changes: 1 addition & 1 deletion test/other/codesize/test_codesize_files_js_fs.gzsize
Original file line number Diff line number Diff line change
@@ -1 +1 @@
7638
7656
2 changes: 1 addition & 1 deletion test/other/codesize/test_codesize_files_js_fs.jssize
Original file line number Diff line number Diff line change
@@ -1 +1 @@
18780
18842
2 changes: 1 addition & 1 deletion test/other/codesize/test_codesize_files_wasmfs.gzsize
Original file line number Diff line number Diff line change
@@ -1 +1 @@
2866
2882
2 changes: 1 addition & 1 deletion test/other/codesize/test_codesize_files_wasmfs.jssize
Original file line number Diff line number Diff line change
@@ -1 +1 @@
6140
6202
1 change: 0 additions & 1 deletion test/other/codesize/test_codesize_hello_O0.exports
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@ __indirect_function_table
__wasm_call_ctors
_emscripten_stack_alloc
_emscripten_stack_restore
dynCall_jiji
emscripten_stack_get_base
emscripten_stack_get_current
emscripten_stack_get_end
Expand Down
4 changes: 0 additions & 4 deletions test/other/codesize/test_codesize_hello_O0.funcs
Original file line number Diff line number Diff line change
Expand Up @@ -25,9 +25,6 @@ $__wasi_syscall_ret
$__wasm_call_ctors
$_emscripten_stack_alloc
$_emscripten_stack_restore
$_emscripten_tempret_get
$_emscripten_tempret_set
$dynCall_jiji
$emscripten_stack_get_base
$emscripten_stack_get_current
$emscripten_stack_get_end
Expand All @@ -42,7 +39,6 @@ $frexp
$getint
$getpid
$init_pthread_self
$legalstub$dynCall_jiji
$main
$memchr
$out
Expand Down
2 changes: 1 addition & 1 deletion test/other/codesize/test_codesize_hello_O0.gzsize
Original file line number Diff line number Diff line change
@@ -1 +1 @@
8003
8017
2 changes: 1 addition & 1 deletion test/other/codesize/test_codesize_hello_O0.jssize
Original file line number Diff line number Diff line change
@@ -1 +1 @@
21535
21597
2 changes: 1 addition & 1 deletion test/other/codesize/test_codesize_hello_O0.size
Original file line number Diff line number Diff line change
@@ -1 +1 @@
14531
15142
1 change: 0 additions & 1 deletion test/other/codesize/test_codesize_hello_O1.exports
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@ __indirect_function_table
__wasm_call_ctors
_emscripten_stack_alloc
_emscripten_stack_restore
dynCall_jiji
emscripten_stack_get_current
main
memory
4 changes: 0 additions & 4 deletions test/other/codesize/test_codesize_hello_O1.funcs
Original file line number Diff line number Diff line change
Expand Up @@ -13,13 +13,9 @@ $__wasi_syscall_ret
$__wasm_call_ctors
$_emscripten_stack_alloc
$_emscripten_stack_restore
$_emscripten_tempret_get
$_emscripten_tempret_set
$dynCall_jiji
$emscripten_stack_get_current
$fputs
$fwrite
$legalstub$dynCall_jiji
$main
$puts
$strlen
2 changes: 1 addition & 1 deletion test/other/codesize/test_codesize_hello_O1.gzsize
Original file line number Diff line number Diff line change
@@ -1 +1 @@
2713
2697
2 changes: 1 addition & 1 deletion test/other/codesize/test_codesize_hello_O1.jssize
Original file line number Diff line number Diff line change
@@ -1 +1 @@
6921
6911
2 changes: 1 addition & 1 deletion test/other/codesize/test_codesize_hello_O1.size
Original file line number Diff line number Diff line change
@@ -1 +1 @@
2535
2634
1 change: 0 additions & 1 deletion test/other/codesize/test_codesize_hello_O2.exports
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@ __indirect_function_table
__wasm_call_ctors
_emscripten_stack_alloc
_emscripten_stack_restore
dynCall_jiji
emscripten_stack_get_current
main
memory
Loading
Loading