Skip to content

Commit 5e9baa6

Browse files
authored
Restore test_other.other.test_codesize_cxx_lto after Binaryen fix. NFC (#24783)
Followup to https://github.com/emscripten-core/emscripten/pull/24777/files#diff-59f6b346090f49748193fef565b0c7e223f93ecef5bb116fa370d0b864af9493 after WebAssembly/binaryen#7748 landed. Code size savings are from WebAssembly/binaryen#7728
1 parent 71ec929 commit 5e9baa6

File tree

2 files changed

+31
-34
lines changed

2 files changed

+31
-34
lines changed
Lines changed: 30 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -1,42 +1,40 @@
11
{
2-
"a.out.js": 19506,
3-
"a.out.js.gz": 8014,
4-
"a.out.nodebug.wasm": 121757,
5-
"a.out.nodebug.wasm.gz": 48787,
6-
"total": 141263,
7-
"total_gz": 56801,
2+
"a.out.js": 19130,
3+
"a.out.js.gz": 7843,
4+
"a.out.nodebug.wasm": 106474,
5+
"a.out.nodebug.wasm.gz": 42624,
6+
"total": 125604,
7+
"total_gz": 50467,
88
"sent": [
99
"a (emscripten_resize_heap)",
10-
"b (_tzset_js)",
11-
"c (_setitimer_js)",
12-
"d (_emscripten_runtime_keepalive_clear)",
13-
"e (_abort_js)",
14-
"f (proc_exit)",
15-
"g (fd_write)",
16-
"h (fd_seek)",
17-
"i (fd_read)",
18-
"j (fd_close)",
19-
"k (environ_sizes_get)",
20-
"l (environ_get)"
10+
"b (_setitimer_js)",
11+
"c (_emscripten_runtime_keepalive_clear)",
12+
"d (_abort_js)",
13+
"e (proc_exit)",
14+
"f (fd_write)",
15+
"g (fd_seek)",
16+
"h (fd_read)",
17+
"i (fd_close)",
18+
"j (environ_sizes_get)",
19+
"k (environ_get)"
2120
],
2221
"imports": [
2322
"a (emscripten_resize_heap)",
24-
"b (_tzset_js)",
25-
"c (_setitimer_js)",
26-
"d (_emscripten_runtime_keepalive_clear)",
27-
"e (_abort_js)",
28-
"f (proc_exit)",
29-
"g (fd_write)",
30-
"h (fd_seek)",
31-
"i (fd_read)",
32-
"j (fd_close)",
33-
"k (environ_sizes_get)",
34-
"l (environ_get)"
23+
"b (_setitimer_js)",
24+
"c (_emscripten_runtime_keepalive_clear)",
25+
"d (_abort_js)",
26+
"e (proc_exit)",
27+
"f (fd_write)",
28+
"g (fd_seek)",
29+
"h (fd_read)",
30+
"i (fd_close)",
31+
"j (environ_sizes_get)",
32+
"k (environ_get)"
3533
],
3634
"exports": [
37-
"m (memory)",
38-
"n (__wasm_call_ctors)",
39-
"o (main)",
40-
"p (_emscripten_timeout)"
35+
"l (memory)",
36+
"m (__wasm_call_ctors)",
37+
"n (main)",
38+
"o (_emscripten_timeout)"
4139
]
4240
}

test/test_other.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9377,8 +9377,7 @@ def test_codesize_minimal_pthreads(self, args):
93779377
'ctors1': (['-O2', '-sEVAL_CTORS'],),
93789378
'ctors2': (['-O2', '-sEVAL_CTORS=2'],),
93799379
'wasmfs': (['-O2', '-sWASMFS'],),
9380-
# Disabled until https://github.com/WebAssembly/binaryen/pull/7748 lands
9381-
#'lto': (['-Oz', '-flto'],),
9380+
'lto': (['-Oz', '-flto'],),
93829381
})
93839382
def test_codesize_cxx(self, args):
93849383
# do not check functions in this test as there are a lot of libc++ functions

0 commit comments

Comments
 (0)