Skip to content

Commit 8f5936b

Browse files
kripkenBeuc
authored andcommitted
Disable lsr in the wasm backend [ci skip] (emscripten-core#7386)
I did another round of tests now to follow up on WebAssembly/binaryen#1054 , comparing v8 and sm, on the wasm backend with and without lsr. Disabling lsr reduces code size in all the benchmark suite, up to 1% (but usually less). It's also somewhat faster (5-10%) in a few microbenchmarks, on both VMs (so this is not a VM-specific issue). So seems worthwhile. LLVM bug: https://bugs.llvm.org/show_bug.cgi?id=39488
1 parent 9d34a0e commit 8f5936b

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

tools/shared.py

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1876,6 +1876,11 @@ def llvm_backend_args():
18761876

18771877
# asm.js-style setjmp/longjmp handling
18781878
args += ['-enable-emscripten-sjlj']
1879+
1880+
# better (smaller, sometimes faster) codegen, see binaryen#1054
1881+
# and https://bugs.llvm.org/show_bug.cgi?id=39488
1882+
args += ['-disable-lsr']
1883+
18791884
return args
18801885

18811886
@staticmethod

0 commit comments

Comments
 (0)