refine precision limits on unsafe_math (#3589)

mishoo · Nov 16, 2019 · d1b2ece · d1b2ece
1 parent 552be61
commit d1b2ece
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 1 deletion.
diff --git a/lib/compress.js b/lib/compress.js
@@ -2953,7 +2953,8 @@ merge(Compressor.prototype, {
                 && typeof result == "number"
                 && (this.operator == "+" || this.operator == "-")) {
                 var digits = Math.max(0, decimals(left), decimals(right));
-                if (digits < 21) return +result.toFixed(digits);
+                // 53-bit significand => 15.95 decimal places
+                if (digits < 16) return +result.toFixed(digits);
             }
             return result;
 

diff --git a/test/compress/numbers.js b/test/compress/numbers.js
@@ -917,3 +917,17 @@ issue_3547_4: {
         "number 0",
     ]
 }
+
+unsafe_math_rounding: {
+    options = {
+        evaluate: true,
+        unsafe_math: true,
+    }
+    input: {
+        console.log(4 / -3 + 1 === 1 / -3);
+    }
+    expect: {
+        console.log(false);
+    }
+    expect_stdout: "false"
+}