fix shift computation in getMaxBits - in wasm only the lower 5 bits matter for a 32-bit shift

2 files changed, 20 insertions, 2 deletions

diff --git a/test/passes/optimize-instructions.txt b/test/passes/optimize-instructions.txt
index 301a6cfc0..68359d9fb 100644
--- a/test/passes/optimize-instructions.txt
+++ b/test/passes/optimize-instructions.txt
@@ -805,7 +805,7 @@
      (i32.const -1)
      (i32.const 2147483647)
     )
-    (i32.const 32)
+    (i32.const 31)
    )
   )
   (drop
@@ -2006,4 +2006,13 @@
    (i32.const 1)
   )
  )
+ (func $neg-shifts-and-255 (type $2) (result i32)
+  (i32.and
+   (i32.shr_u
+    (i32.const -99)
+    (i32.const -32)
+   )
+   (i32.const 255)
+  )
+ )
 )
diff --git a/test/passes/optimize-instructions.wast b/test/passes/optimize-instructions.wast
index cf8275412..5300b8da8 100644
--- a/test/passes/optimize-instructions.wast
+++ b/test/passes/optimize-instructions.wast
@@ -842,7 +842,7 @@
               (i32.const -1)
               (i32.const 2147483647)
             )
-            (i32.const 32)
+            (i32.const 31) ;; adjusted after we fixed shift computation to just look at lower 5 bits
           )
           (i32.const 24)
         )
@@ -2433,4 +2433,13 @@
     (i32.const 1)
    )
   )
+  (func $neg-shifts-and-255 (result i32)
+    (i32.and
+     (i32.shr_u
+      (i32.const -99)
+      (i32.const -32) ;; this shift does nothing
+     )
+     (i32.const 255)
+    )
+  )
 )