diff options
| author | Alon Zakai <azakai@google.com> | 2021-04-20 08:21:42 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-04-20 08:21:42 -0700 |
| commit | 02ae2661c9cb748d5479017a3db7bbd222e2daf5 (patch) | |
| tree | 3446624c393b21ab349c654189fa2aa49d84cb87 /test/lit/passes/optimize-instructions.wast | |
| parent | f180f6c11d2ddf9acc806333c0a452bb57f8acf9 (diff) | |
| download | binaryen-02ae2661c9cb748d5479017a3db7bbd222e2daf5.tar.gz binaryen-02ae2661c9cb748d5479017a3db7bbd222e2daf5.tar.bz2 binaryen-02ae2661c9cb748d5479017a3db7bbd222e2daf5.zip | |
Optimize if/select with one arm an EqZ and another a 0 or a 1 (#3822)
(select
(i32.eqz (X))
(i32.const 0|1)
(Y)
)
=>
(i32.eqz
(select
(X)
(i32.const 1|0)
(Y)
)
)
This is beneficial as the eqz may be folded into something on the outside. I
see this pattern in real-world code, both a GC benchmark (which is why I
noticed it) and it shrinks code size by tiny amounts on the emscripten
benchmark suite as well.
Diffstat (limited to 'test/lit/passes/optimize-instructions.wast')
| -rw-r--r-- | test/lit/passes/optimize-instructions.wast | 117 |
1 files changed, 117 insertions, 0 deletions
diff --git a/test/lit/passes/optimize-instructions.wast b/test/lit/passes/optimize-instructions.wast index 5a180e3a5..bb05f17ba 100644 --- a/test/lit/passes/optimize-instructions.wast +++ b/test/lit/passes/optimize-instructions.wast @@ -11701,4 +11701,121 @@ (f64.abs (f64.add (local.get $x0) (local.get $x0))) )) ) + ;; CHECK: (func $ternary (param $x i32) (param $y i32) + ;; CHECK-NEXT: (drop + ;; CHECK-NEXT: (i32.eqz + ;; CHECK-NEXT: (select + ;; CHECK-NEXT: (i32.const 1) + ;; CHECK-NEXT: (local.get $y) + ;; CHECK-NEXT: (local.get $x) + ;; CHECK-NEXT: ) + ;; CHECK-NEXT: ) + ;; CHECK-NEXT: ) + ;; CHECK-NEXT: (drop + ;; CHECK-NEXT: (i32.eqz + ;; CHECK-NEXT: (select + ;; CHECK-NEXT: (i32.const 0) + ;; CHECK-NEXT: (local.get $y) + ;; CHECK-NEXT: (local.get $x) + ;; CHECK-NEXT: ) + ;; CHECK-NEXT: ) + ;; CHECK-NEXT: ) + ;; CHECK-NEXT: (drop + ;; CHECK-NEXT: (i32.eqz + ;; CHECK-NEXT: (select + ;; CHECK-NEXT: (local.get $y) + ;; CHECK-NEXT: (i32.const 1) + ;; CHECK-NEXT: (local.get $x) + ;; CHECK-NEXT: ) + ;; CHECK-NEXT: ) + ;; CHECK-NEXT: ) + ;; CHECK-NEXT: (drop + ;; CHECK-NEXT: (i32.eqz + ;; CHECK-NEXT: (select + ;; CHECK-NEXT: (local.get $y) + ;; CHECK-NEXT: (i32.const 0) + ;; CHECK-NEXT: (local.get $x) + ;; CHECK-NEXT: ) + ;; CHECK-NEXT: ) + ;; CHECK-NEXT: ) + ;; CHECK-NEXT: (drop + ;; CHECK-NEXT: (i32.eqz + ;; CHECK-NEXT: (if (result i32) + ;; CHECK-NEXT: (local.get $x) + ;; CHECK-NEXT: (local.get $y) + ;; CHECK-NEXT: (i32.const 0) + ;; CHECK-NEXT: ) + ;; CHECK-NEXT: ) + ;; CHECK-NEXT: ) + ;; CHECK-NEXT: ) + (func $ternary (param $x i32) (param $y i32) + (drop + (select + (i32.const 0) + (i32.eqz + (local.get $y) + ) + (local.get $x) + ) + ) + (drop + (select + (i32.const 1) + (i32.eqz + (local.get $y) + ) + (local.get $x) + ) + ) + (drop + (select + (i32.eqz + (local.get $y) + ) + (i32.const 0) + (local.get $x) + ) + ) + (drop + (select + (i32.eqz + (local.get $y) + ) + (i32.const 1) + (local.get $x) + ) + ) + ;; if works too + (drop + (if (result i32) + (local.get $x) + (i32.eqz + (local.get $y) + ) + (i32.const 1) + ) + ) + ) + ;; CHECK: (func $ternary-no (param $x i32) (param $y i32) + ;; CHECK-NEXT: (drop + ;; CHECK-NEXT: (select + ;; CHECK-NEXT: (i32.const 2) + ;; CHECK-NEXT: (i32.eqz + ;; CHECK-NEXT: (local.get $y) + ;; CHECK-NEXT: ) + ;; CHECK-NEXT: (local.get $x) + ;; CHECK-NEXT: ) + ;; CHECK-NEXT: ) + ;; CHECK-NEXT: ) + (func $ternary-no (param $x i32) (param $y i32) + (drop + (select + (i32.const 2) ;; only 0 and 1 work + (i32.eqz + (local.get $y) + ) + (local.get $x) + ) + ) + ) ) |