enhance hoist_vars (#5865)

alexlamsl · web-flow · commit ce8ef52e2bca · 2024-06-25T23:56:39.000+03:00
diff --git a/lib/compress.js b/lib/compress.js
@@ -10353,7 +10353,6 @@ Compressor.prototype.compress = function(node) {
                 def.assignments++;
             }
             def.eliminated++;
-            def.single_use = false;
             return a;
         }, []);
         if (assignments.length == 0) return null;
diff --git a/test/compress/hoist_vars.js b/test/compress/hoist_vars.js
@@ -228,6 +228,7 @@ issue_4489: {
         evaluate: true,
         hoist_vars: true,
         reduce_vars: true,
+        sequences: true,
         toplevel: true,
         unused: true,
     }
@@ -363,6 +364,7 @@ issue_4893_1: {
         evaluate: true,
         hoist_vars: true,
         reduce_vars: true,
+        side_effects: true,
         toplevel: true,
         unused: true,
     }
@@ -383,9 +385,8 @@ issue_4893_1: {
     }
     expect: {
         try{
-            (function f() {
+            (function() {
                 null.p += 42;
-                f;
             })();
         } catch (e) {
             console.log("PASS");
@@ -422,9 +423,7 @@ issue_4893_2: {
     expect: {
         try{
             (function() {
-                var a;
-                a = null;
-                a.p += 42;
+                null.p += 42;
             })();
         } catch (e) {
             console.log("PASS");
@@ -610,6 +609,7 @@ issue_5411_2: {
         evaluate: true,
         hoist_vars: true,
         reduce_vars: true,
+        sequences: true,
         toplevel: true,
         unused: true,
     }
@@ -622,9 +622,9 @@ issue_5411_2: {
     }
     expect: {
         var b, c;
-        b++;
+        b++,
         b = "PASS",
-        c;
+        c,
         console.log(b);
     }
     expect_stdout: "PASS"

Original file line number	Diff line number	Diff line change
`@@ -10353,7 +10353,6 @@ Compressor.prototype.compress = function(node) {`
`10353`	`10353`	`def.assignments++;`
`10354`	`10354`	`}`
`10355`	`10355`	`def.eliminated++;`
`10356`		`- def.single_use = false;`
`10357`	`10356`	`return a;`
`10358`	`10357`	`}, []);`
`10359`	`10358`	`if (assignments.length == 0) return null;`