fix corner case in evaluate (#3685)

fixes #3684
mishoo · Jan 15, 2020 · 41a6eb8 · 41a6eb8
1 parent 91d87ae
commit 41a6eb8
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 1 deletion.
diff --git a/lib/compress.js b/lib/compress.js
@@ -2497,9 +2497,10 @@ merge(Compressor.prototype, {
                     && (!(right instanceof AST_Constant) || right.value == 0);
               case "&&":
               case "||":
-              case "*":
                 return left.is_negative_zero() || right.is_negative_zero();
+              case "*":
               case "/":
+                return true;
               case "%":
                 return left.is_negative_zero();
               default:

diff --git a/test/compress/numbers.js b/test/compress/numbers.js
@@ -1215,3 +1215,21 @@ issue_3682_3: {
     }
     expect_stdout: "-Infinity"
 }
+
+issue_3684: {
+    options = {
+        evaluate: true,
+    }
+    input: {
+        console.log(1 / (-1 * (0 & console) + 0));
+        console.log(1 / ((0 & console) / -1 + 0));
+    }
+    expect: {
+        console.log(1 / (-1 * (0 & console) + 0));
+        console.log(1 / ((0 & console) / -1 + 0));
+    }
+    expect_stdout: [
+        "Infinity",
+        "Infinity",
+    ]
+}