diff options
| -rw-r--r-- | lib/compress.js | 12 | ||||
| -rw-r--r-- | test/compress/hoist_props.js | 24 |
2 files changed, 35 insertions, 1 deletions
diff --git a/lib/compress.js b/lib/compress.js index 71f2eae1..3bb6cb24 100644 --- a/lib/compress.js +++ b/lib/compress.js @@ -519,7 +519,7 @@ merge(Compressor.prototype, { if (parent instanceof AST_Call && node === parent.expression) return; if (parent instanceof AST_Sequence && node !== parent.tail_node()) return; if (parent instanceof AST_SimpleStatement) return; - if (parent instanceof AST_Unary) return; + if (parent instanceof AST_Unary && !unary_side_effects[parent.operator]) return; d.direct_access = true; } @@ -4067,6 +4067,16 @@ merge(Compressor.prototype, { })); return make_sequence(node, assignments); } + if (node instanceof AST_Unary + && !unary_side_effects[node.operator] + && node.expression instanceof AST_SymbolRef + && node.expression.definition().id in defs_by_id) { + node = node.clone(); + node.expression = make_node(AST_Object, node, { + properties: [] + }); + return node; + } if (node instanceof AST_VarDef && can_hoist(node.name, node.value, 0)) { descend(node, this); var defs = new Dictionary(); diff --git a/test/compress/hoist_props.js b/test/compress/hoist_props.js index 6f309566..b055ae7b 100644 --- a/test/compress/hoist_props.js +++ b/test/compress/hoist_props.js @@ -862,3 +862,27 @@ issue_3071_3: { } expect_stdout: "2" } + +issue_3411: { + options = { + hoist_props: true, + reduce_vars: true, + } + input: { + var c = 1; + !function f() { + var o = { + p: --c && f() + }; + +o || console.log("PASS"); + }(); + } + expect: { + var c = 1; + !function f() { + var o_p = --c && f(); + +{} || console.log("PASS"); + }(); + } + expect_stdout: "PASS" +} |