From 603e90d71c976d01279b4a1ef9ddf5825cec4b28 Mon Sep 17 00:00:00 2001
From: alexlamsl <alexlamsl@gmail.com>
Date: Tue, 17 Oct 2017 04:42:11 +0800
Subject: [PATCH] enhance `collapse_vars` around lazy operations

---
 lib/compress.js                | 11 ++++++-----
 test/compress/collapse_vars.js | 21 +++++++++++++++++++++
 2 files changed, 27 insertions(+), 5 deletions(-)

diff --git a/lib/compress.js b/lib/compress.js
index b89e0df9..374456ac 100644
--- a/lib/compress.js
+++ b/lib/compress.js
@@ -778,6 +778,7 @@ merge(Compressor.prototype, {
                     // Locate symbols which may execute code outside of scanning range
                     var lvalues = get_lvalues(candidate);
                     if (lhs instanceof AST_SymbolRef) lvalues[lhs.name] = false;
+                    var one_off = lhs instanceof AST_Symbol && lhs.definition().references.length == 1;
                     var side_effects = value_has_side_effects(candidate);
                     var hit = candidate.name instanceof AST_SymbolFunarg;
                     var abort = false, replaced = false;
@@ -844,11 +845,11 @@ merge(Compressor.prototype, {
                                    || side_effects && !references_in_scope(node.definition()))
                             || (sym = lhs_or_def(node))
                                 && (sym instanceof AST_PropAccess || sym.name in lvalues)
-                            || parent instanceof AST_Binary && lazy_op(parent.operator)
-                            || parent instanceof AST_Case
-                            || parent instanceof AST_Conditional
-                            || parent instanceof AST_For
-                            || parent instanceof AST_If) {
+                            || (side_effects || !one_off)
+                                && (parent instanceof AST_Binary && lazy_op(parent.operator)
+                                    || parent instanceof AST_Case
+                                    || parent instanceof AST_Conditional
+                                    || parent instanceof AST_If)) {
                             if (!(node instanceof AST_Scope)) descend(node, tt);
                             abort = true;
                             return node;
diff --git a/test/compress/collapse_vars.js b/test/compress/collapse_vars.js
index 13fff97a..b66efbbc 100644
--- a/test/compress/collapse_vars.js
+++ b/test/compress/collapse_vars.js
@@ -2951,3 +2951,24 @@ issue_2364_9: {
     }
     expect_stdout: "PASS"
 }
+
+pure_getters_chain: {
+    options = {
+        collapse_vars: true,
+        pure_getters: true,
+    }
+    input: {
+        function o(t, r) {
+            var a = t[1], s = t[2], o = t[3], i = t[5];
+            return a <= 23 && s <= 59 && o <= 59 && (!r || i);
+        }
+        console.log(o([ , 23, 59, 59, , 42], 1));
+    }
+    expect: {
+        function o(t, r) {
+            return t[1] <= 23 && t[2] <= 59 && t[3] <= 59 && (!r || t[5]);
+        }
+        console.log(o([ , 23, 59, 59, , 42], 1));
+    }
+    expect_stdout: "42"
+}