fix corner case in `collapse_vars` (#3978)

fixes #3976

diff --git a/lib/compress.js b/lib/compress.js
index 907c9e7..cfbafbe 100644 (file)
--- a/lib/compress.js
+++ b/lib/compress.js
@@ -1468,7 +1468,7 @@ merge(Compressor.prototype, {
                 if (node instanceof AST_Call) {
                     var fn = node.expression;
                     if (fn instanceof AST_SymbolRef) {
-                        if (fn.definition().recursive_refs > 0) return true;
+                        if (recursive_ref(compressor, fn.definition())) return true;
                         fn = fn.fixed_value();
                     }
                     if (!(fn instanceof AST_Lambda)) return true;

diff --git a/test/compress/collapse_vars.js b/test/compress/collapse_vars.js
index 71c3a14..dcc95b2 100644 (file)
--- a/test/compress/collapse_vars.js
+++ b/test/compress/collapse_vars.js
@@ -8242,3 +8242,41 @@ issue_3971: {
     }
     expect_stdout: "1"
 }
+
+issue_3976: {
+    options = {
+        collapse_vars: true,
+        conditionals: true,
+        evaluate: true,
+        inline: true,
+        reduce_vars: true,
+        unused: true,
+    }
+    input: {
+        function f() {
+            console.log("FAIL");
+        }
+        (function(a) {
+            function g() {
+                if ((a = 0) || f(0)) {
+                    f();
+                } else {
+                    f();
+                }
+                if (h(a = 0));
+            }
+            function h() {
+                g();
+            }
+        })();
+        console.log("PASS");
+    }
+    expect: {
+        function f() {
+            console.log("FAIL");
+        }
+        void 0;
+        console.log("PASS");
+    }
+    expect_stdout: "PASS"
+}