diff options
| author | Alon Zakai <azakai@google.com> | 2019-08-16 10:16:28 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2019-08-16 10:16:28 -0700 |
| commit | ee0564088c7f89814bf951cc5936aa096538e38f (patch) | |
| tree | d327c8f69b71f785faa0ac55c93662cd55280df9 /test/wasm2js/br_table_hoisting.2asm.js.opt | |
| parent | 1b8c19a021c09aa4024fe79753ab922cd5762ec0 (diff) | |
| download | binaryen-ee0564088c7f89814bf951cc5936aa096538e38f.tar.gz binaryen-ee0564088c7f89814bf951cc5936aa096538e38f.tar.bz2 binaryen-ee0564088c7f89814bf951cc5936aa096538e38f.zip | |
wasm2js: Fix switch lowering, don't fall through after the hoisted parts (#2301)
The switch lowering will "hoist" blocks of code into the JS switch when it can. If it can hoist some but not others, it must not fall through into those others (while it can fall through the hoisted ones - they began as nested blocks with falling-through between them). To fix this, after the hoisted ones issue a break out of the switch (which now contains all the hoisted code, so breaking out of it gets to the code right after the hoisted ones).
fixes #2300
Diffstat (limited to 'test/wasm2js/br_table_hoisting.2asm.js.opt')
| -rw-r--r-- | test/wasm2js/br_table_hoisting.2asm.js.opt | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/test/wasm2js/br_table_hoisting.2asm.js.opt b/test/wasm2js/br_table_hoisting.2asm.js.opt index 44c9d0417..f28214760 100644 --- a/test/wasm2js/br_table_hoisting.2asm.js.opt +++ b/test/wasm2js/br_table_hoisting.2asm.js.opt @@ -100,6 +100,7 @@ function asmFunc(global, env, buffer) { } zed(-1); zed(-2); + break; case 0: break a; case 1: @@ -140,6 +141,7 @@ function asmFunc(global, env, buffer) { } zed(-1); zed(-2); + break; case 0: break a; case 1: |