From: Alex Lam S.L <alexlamsl@gmail.com>
Date: Mon, 18 Jan 2021 22:34:48 +0000 (+0000)
Subject: fix corner case in `reduce_vars` (#4569)
X-Git-Url: https://git.ndcode.org/public/gitweb.cgi?a=commitdiff_plain;h=b57bae4b9e528ec7aeea7fd5768295a559cb5e5d;p=UglifyJS.git

fix corner case in `reduce_vars` (#4569)

fixes #4568
---

diff --git a/lib/compress.js b/lib/compress.js
index 81511535..e33f4f3f 100644
--- a/lib/compress.js
+++ b/lib/compress.js
@@ -9733,7 +9733,7 @@ merge(Compressor.prototype, {
                         single_use = false;
                     }
                     if (single_use) fixed.parent_scope = self.scope;
-                } else if (!fixed || !fixed.is_constant_expression()) {
+                } else if (!fixed || !fixed.is_constant_expression() || fixed.drop_side_effect_free(compressor)) {
                     single_use = false;
                 }
             }
diff --git a/test/compress/ie8.js b/test/compress/ie8.js
index c28512ac..ed3950a0 100644
--- a/test/compress/ie8.js
+++ b/test/compress/ie8.js
@@ -2900,3 +2900,22 @@ issue_4250: {
     }
     expect_stdout: "0"
 }
+
+issue_4568: {
+    options = {
+        ie8: true,
+        reduce_vars: true,
+        unused: true,
+    }
+    input: {
+        console.log(typeof f, function(a) {
+            return a.length;
+        }([ function f() {} ]));
+    }
+    expect: {
+        console.log(typeof f, function(a) {
+            return a.length;
+        }([ function f() {} ]));
+    }
+    expect_stdout: "undefined 1"
+}
diff --git a/test/compress/reduce_vars.js b/test/compress/reduce_vars.js
index ea9efee0..185a39cf 100644
--- a/test/compress/reduce_vars.js
+++ b/test/compress/reduce_vars.js
@@ -7601,3 +7601,32 @@ issue_4188_2: {
     }
     expect_stdout: "number undefined"
 }
+
+issue_4568: {
+    options = {
+        booleans: true,
+        conditionals: true,
+        dead_code: true,
+        evaluate: true,
+        loops: true,
+        passes: 2,
+        reduce_vars: true,
+        sequences: true,
+        unused: true,
+    }
+    input: {
+        (function(a) {
+            a && console.log("FAIL");
+            if (1)
+                do {
+                    if (!console.log("PASS")) break;
+                } while (1);
+        })(!(0 !== delete NaN));
+    }
+    expect: {
+        (function(a) {
+            for (a && console.log("FAIL"), 1; console.log("PASS"); ) 1;
+        })(!(0 !== delete NaN));
+    }
+    expect_stdout: "PASS"
+}