diff options
| author | Alon Zakai <azakai@google.com> | 2020-11-01 12:00:28 -0800 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2020-11-01 12:00:28 -0800 |
| commit | 69367c09b2e26701ff4d99fec0e462c0114aac87 (patch) | |
| tree | bfcee1c4be7b4798bea873396b8b7c33b3d24b65 /test/wasm2js/i64-add-sub.2asm.js.opt | |
| parent | f1f2843699f5fd7b87dcefe00ce1eb8f72b37465 (diff) | |
| download | binaryen-69367c09b2e26701ff4d99fec0e462c0114aac87.tar.gz binaryen-69367c09b2e26701ff4d99fec0e462c0114aac87.tar.bz2 binaryen-69367c09b2e26701ff4d99fec0e462c0114aac87.zip | |
RemoveUnusedBrs: Properly check for effects in selectify() (#3310)
Selectify turns an if-else into a select where possible. Previously we abandoned
hope if any part of the if had a side effect. But it's fine for the condition to have a
side effect, so long as moving it to the end doesn't invalidate the arms.
Diffstat (limited to 'test/wasm2js/i64-add-sub.2asm.js.opt')
| -rw-r--r-- | test/wasm2js/i64-add-sub.2asm.js.opt | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/test/wasm2js/i64-add-sub.2asm.js.opt b/test/wasm2js/i64-add-sub.2asm.js.opt index 6d00c9321..d44e7657d 100644 --- a/test/wasm2js/i64-add-sub.2asm.js.opt +++ b/test/wasm2js/i64-add-sub.2asm.js.opt @@ -21,9 +21,7 @@ function asmFunc(global, env) { $5 = $5 | 0; $1_1 = $1_1 + $3 | 0; $0 = $0 + $2 | 0; - if ($2 >>> 0 > $0 >>> 0) { - $1_1 = $1_1 + 1 | 0 - } + $1_1 = $2 >>> 0 > $0 >>> 0 ? $1_1 + 1 | 0 : $1_1; return ($0 | 0) == ($4 | 0) & ($1_1 | 0) == ($5 | 0); } |