Merge branch 'master' into harmony-v2.8.21
This commit is contained in:
305
lib/compress.js
305
lib/compress.js
@@ -48,43 +48,45 @@ function Compressor(options, false_by_default) {
|
||||
return new Compressor(options, false_by_default);
|
||||
TreeTransformer.call(this, this.before, this.after);
|
||||
this.options = defaults(options, {
|
||||
sequences : !false_by_default,
|
||||
properties : !false_by_default,
|
||||
angular : false,
|
||||
booleans : !false_by_default,
|
||||
cascade : !false_by_default,
|
||||
collapse_vars : !false_by_default,
|
||||
comparisons : !false_by_default,
|
||||
conditionals : !false_by_default,
|
||||
dead_code : !false_by_default,
|
||||
drop_console : false,
|
||||
drop_debugger : !false_by_default,
|
||||
ecma : 5,
|
||||
evaluate : !false_by_default,
|
||||
expression : false,
|
||||
global_defs : {},
|
||||
hoist_funs : !false_by_default,
|
||||
hoist_vars : false,
|
||||
if_return : !false_by_default,
|
||||
join_vars : !false_by_default,
|
||||
keep_fargs : true,
|
||||
keep_fnames : false,
|
||||
keep_infinity : false,
|
||||
loops : !false_by_default,
|
||||
negate_iife : !false_by_default,
|
||||
passes : 1,
|
||||
properties : !false_by_default,
|
||||
pure_getters : false,
|
||||
pure_funcs : null,
|
||||
reduce_vars : !false_by_default,
|
||||
screw_ie8 : true,
|
||||
sequences : !false_by_default,
|
||||
side_effects : !false_by_default,
|
||||
switches : !false_by_default,
|
||||
top_retain : null,
|
||||
toplevel : !!(options && options["top_retain"]),
|
||||
unsafe : false,
|
||||
unsafe_comps : false,
|
||||
unsafe_math : false,
|
||||
unsafe_proto : false,
|
||||
conditionals : !false_by_default,
|
||||
comparisons : !false_by_default,
|
||||
evaluate : !false_by_default,
|
||||
booleans : !false_by_default,
|
||||
loops : !false_by_default,
|
||||
unused : !false_by_default,
|
||||
toplevel : !!(options && options["top_retain"]),
|
||||
top_retain : null,
|
||||
hoist_funs : !false_by_default,
|
||||
keep_fargs : true,
|
||||
keep_fnames : false,
|
||||
hoist_vars : false,
|
||||
if_return : !false_by_default,
|
||||
join_vars : !false_by_default,
|
||||
collapse_vars : !false_by_default,
|
||||
reduce_vars : !false_by_default,
|
||||
cascade : !false_by_default,
|
||||
side_effects : !false_by_default,
|
||||
pure_getters : false,
|
||||
pure_funcs : null,
|
||||
negate_iife : !false_by_default,
|
||||
screw_ie8 : true,
|
||||
ecma : 5,
|
||||
drop_console : false,
|
||||
angular : false,
|
||||
expression : false,
|
||||
warnings : true,
|
||||
global_defs : {},
|
||||
passes : 1,
|
||||
}, true);
|
||||
var pure_funcs = this.options["pure_funcs"];
|
||||
if (typeof pure_funcs == "function") {
|
||||
@@ -216,7 +218,12 @@ merge(Compressor.prototype, {
|
||||
}) : make_node(AST_EmptyStatement, node);
|
||||
}
|
||||
return make_node(AST_SimpleStatement, node, {
|
||||
body: node.value || make_node(AST_Undefined, node)
|
||||
body: node.value || make_node(AST_UnaryPrefix, node, {
|
||||
operator: "void",
|
||||
expression: make_node(AST_Number, node, {
|
||||
value: 0
|
||||
})
|
||||
})
|
||||
});
|
||||
}
|
||||
if (node instanceof AST_Lambda && node !== self) {
|
||||
@@ -407,6 +414,18 @@ merge(Compressor.prototype, {
|
||||
}
|
||||
});
|
||||
|
||||
function find_variable(compressor, name) {
|
||||
var scope, i = 0;
|
||||
while (scope = compressor.parent(i++)) {
|
||||
if (scope instanceof AST_Scope) break;
|
||||
if (scope instanceof AST_Catch) {
|
||||
scope = scope.argname.definition().scope;
|
||||
break;
|
||||
}
|
||||
}
|
||||
return scope.find_variable(name);
|
||||
}
|
||||
|
||||
function make_node(ctor, orig, props) {
|
||||
if (!props) props = {};
|
||||
if (orig) {
|
||||
@@ -1071,7 +1090,7 @@ merge(Compressor.prototype, {
|
||||
stat.value = cons_seq(stat.value);
|
||||
}
|
||||
else if (stat instanceof AST_Exit) {
|
||||
stat.value = cons_seq(make_node(AST_Undefined, stat));
|
||||
stat.value = cons_seq(make_node(AST_Undefined, stat).transform(compressor));
|
||||
}
|
||||
else if (stat instanceof AST_Switch) {
|
||||
stat.expression = cons_seq(stat.expression);
|
||||
@@ -1146,8 +1165,12 @@ merge(Compressor.prototype, {
|
||||
}));
|
||||
};
|
||||
|
||||
function is_undefined(node) {
|
||||
return node instanceof AST_Undefined || node.is_undefined;
|
||||
function is_undefined(node, compressor) {
|
||||
return node.is_undefined
|
||||
|| node instanceof AST_Undefined
|
||||
|| node instanceof AST_UnaryPrefix
|
||||
&& node.operator == "void"
|
||||
&& !node.expression.has_side_effects(compressor);
|
||||
}
|
||||
|
||||
/* -----[ boolean/negation helpers ]----- */
|
||||
@@ -1339,7 +1362,7 @@ merge(Compressor.prototype, {
|
||||
return this;
|
||||
}
|
||||
});
|
||||
var unaryPrefix = makePredicate("! ~ - +");
|
||||
var unaryPrefix = makePredicate("! ~ - + void");
|
||||
AST_Node.DEFMETHOD("is_constant", function(){
|
||||
// Accomodate when compress option evaluate=false
|
||||
// as well as the common constant expressions !0 and -1
|
||||
@@ -2638,6 +2661,7 @@ merge(Compressor.prototype, {
|
||||
});
|
||||
|
||||
OPT(AST_Switch, function(self, compressor){
|
||||
if (!compressor.option("switches")) return self;
|
||||
var branch;
|
||||
var value = self.expression.evaluate(compressor);
|
||||
if (value !== self.expression) {
|
||||
@@ -2649,49 +2673,39 @@ merge(Compressor.prototype, {
|
||||
var body = [];
|
||||
var default_branch;
|
||||
var exact_match;
|
||||
var fallthrough;
|
||||
for (var i = 0, len = self.body.length; i < len && !exact_match; i++) {
|
||||
branch = self.body[i];
|
||||
if (branch instanceof AST_Default) {
|
||||
if (!default_branch) default_branch = branch;
|
||||
else if (!fallthrough) {
|
||||
extract_declarations_from_unreachable_code(compressor, branch, decl);
|
||||
continue;
|
||||
if (!default_branch) {
|
||||
default_branch = branch;
|
||||
} else {
|
||||
eliminate_branch(branch, body[body.length - 1]);
|
||||
}
|
||||
} else if (value !== self.expression) {
|
||||
var exp = branch.expression.evaluate(compressor);
|
||||
if (exp === value) {
|
||||
exact_match = branch;
|
||||
if (default_branch) {
|
||||
body.splice(body.indexOf(default_branch), 1);
|
||||
extract_declarations_from_unreachable_code(compressor, default_branch, decl);
|
||||
var default_index = body.indexOf(default_branch);
|
||||
body.splice(default_index, 1);
|
||||
eliminate_branch(default_branch, body[default_index - 1]);
|
||||
default_branch = null;
|
||||
}
|
||||
} else if (exp !== branch.expression && !fallthrough) {
|
||||
extract_declarations_from_unreachable_code(compressor, branch, decl);
|
||||
} else if (exp !== branch.expression) {
|
||||
eliminate_branch(branch, body[body.length - 1]);
|
||||
continue;
|
||||
}
|
||||
}
|
||||
if (aborts(branch)) {
|
||||
if (body.length > 0 && !fallthrough) {
|
||||
var prev = body[body.length - 1];
|
||||
if (prev.body.length == branch.body.length
|
||||
&& make_node(AST_BlockStatement, prev, prev).equivalent_to(make_node(AST_BlockStatement, branch, branch)))
|
||||
prev.body = [];
|
||||
var prev = body[body.length - 1];
|
||||
if (aborts(prev) && prev.body.length == branch.body.length
|
||||
&& make_node(AST_BlockStatement, prev, prev).equivalent_to(make_node(AST_BlockStatement, branch, branch))) {
|
||||
prev.body = [];
|
||||
}
|
||||
body.push(branch);
|
||||
fallthrough = false;
|
||||
} else {
|
||||
body.push(branch);
|
||||
fallthrough = true;
|
||||
}
|
||||
body.push(branch);
|
||||
}
|
||||
for (; i < len && fallthrough; i++) {
|
||||
branch = self.body[i];
|
||||
exact_match.body = exact_match.body.concat(branch.body);
|
||||
fallthrough = !aborts(exact_match);
|
||||
}
|
||||
while (i < len) extract_declarations_from_unreachable_code(compressor, self.body[i++], decl);
|
||||
while (i < len) eliminate_branch(self.body[i++], body[body.length - 1]);
|
||||
if (body.length > 0) {
|
||||
body[0].body = decl.concat(body[0].body);
|
||||
}
|
||||
@@ -2721,9 +2735,25 @@ merge(Compressor.prototype, {
|
||||
has_break = true;
|
||||
});
|
||||
self.walk(tw);
|
||||
if (!has_break) return make_node(AST_BlockStatement, self, body[0]).optimize(compressor);
|
||||
if (!has_break) {
|
||||
body = body[0].body.slice();
|
||||
body.unshift(make_node(AST_SimpleStatement, self.expression, {
|
||||
body: self.expression
|
||||
}));
|
||||
return make_node(AST_BlockStatement, self, {
|
||||
body: body
|
||||
}).optimize(compressor);
|
||||
}
|
||||
}
|
||||
return self;
|
||||
|
||||
function eliminate_branch(branch, prev) {
|
||||
if (prev && !aborts(prev)) {
|
||||
prev.body = prev.body.concat(branch.body);
|
||||
} else {
|
||||
extract_declarations_from_unreachable_code(compressor, branch, decl);
|
||||
}
|
||||
}
|
||||
});
|
||||
|
||||
OPT(AST_Try, function(self, compressor){
|
||||
@@ -3036,7 +3066,7 @@ merge(Compressor.prototype, {
|
||||
if (name instanceof AST_SymbolRef
|
||||
&& name.name == "console"
|
||||
&& name.undeclared()) {
|
||||
return make_node(AST_Undefined, self).transform(compressor);
|
||||
return make_node(AST_Undefined, self).optimize(compressor);
|
||||
}
|
||||
}
|
||||
}
|
||||
@@ -3112,7 +3142,7 @@ merge(Compressor.prototype, {
|
||||
}
|
||||
}
|
||||
}
|
||||
if (is_undefined(self.cdr)) {
|
||||
if (is_undefined(self.cdr, compressor)) {
|
||||
return make_node(AST_UnaryPrefix, self, {
|
||||
operator : "void",
|
||||
expression : self.car
|
||||
@@ -3151,7 +3181,7 @@ merge(Compressor.prototype, {
|
||||
self.expression = e;
|
||||
return self;
|
||||
} else {
|
||||
return make_node(AST_Undefined, self).transform(compressor);
|
||||
return make_node(AST_Undefined, self).optimize(compressor);
|
||||
}
|
||||
}
|
||||
if (compressor.option("booleans") && compressor.in_boolean_context()) {
|
||||
@@ -3175,6 +3205,9 @@ merge(Compressor.prototype, {
|
||||
})).optimize(compressor);
|
||||
}
|
||||
}
|
||||
if (self.operator == "-" && e instanceof AST_Infinity) {
|
||||
e = e.transform(compressor);
|
||||
}
|
||||
if (e instanceof AST_Binary
|
||||
&& (self.operator == "+" || self.operator == "-")
|
||||
&& (e.operator == "*" || e.operator == "/" || e.operator == "%")) {
|
||||
@@ -3184,8 +3217,7 @@ merge(Compressor.prototype, {
|
||||
}
|
||||
// avoids infinite recursion of numerals
|
||||
if (self.operator != "-"
|
||||
|| !(self.expression instanceof AST_Number
|
||||
|| self.expression instanceof AST_Infinity)) {
|
||||
|| !(e instanceof AST_Number || e instanceof AST_Infinity)) {
|
||||
var ev = self.evaluate(compressor);
|
||||
if (ev !== self) {
|
||||
ev = make_node_from_constant(ev, self).optimize(compressor);
|
||||
@@ -3228,8 +3260,8 @@ merge(Compressor.prototype, {
|
||||
|
||||
OPT(AST_Binary, function(self, compressor){
|
||||
function reversible() {
|
||||
return self.left instanceof AST_Constant
|
||||
|| self.right instanceof AST_Constant
|
||||
return self.left.is_constant()
|
||||
|| self.right.is_constant()
|
||||
|| !self.left.has_side_effects(compressor)
|
||||
&& !self.right.has_side_effects(compressor);
|
||||
}
|
||||
@@ -3242,8 +3274,8 @@ merge(Compressor.prototype, {
|
||||
}
|
||||
}
|
||||
if (commutativeOperators(self.operator)) {
|
||||
if (self.right instanceof AST_Constant
|
||||
&& !(self.left instanceof AST_Constant)) {
|
||||
if (self.right.is_constant()
|
||||
&& !self.left.is_constant()) {
|
||||
// if right is a constant, whatever side effects the
|
||||
// left side might have could not influence the
|
||||
// result. hence, force switch.
|
||||
@@ -3281,42 +3313,7 @@ merge(Compressor.prototype, {
|
||||
}
|
||||
break;
|
||||
}
|
||||
if (compressor.option("booleans") && compressor.in_boolean_context()) switch (self.operator) {
|
||||
case "&&":
|
||||
var ll = self.left.evaluate(compressor);
|
||||
var rr = self.right.evaluate(compressor);
|
||||
if (!ll || !rr) {
|
||||
compressor.warn("Boolean && always false [{file}:{line},{col}]", self.start);
|
||||
return make_node(AST_Seq, self, {
|
||||
car: self.left,
|
||||
cdr: make_node(AST_False, self)
|
||||
}).optimize(compressor);
|
||||
}
|
||||
if (ll !== self.left && ll) {
|
||||
return self.right.optimize(compressor);
|
||||
}
|
||||
if (rr !== self.right && rr) {
|
||||
return self.left.optimize(compressor);
|
||||
}
|
||||
break;
|
||||
case "||":
|
||||
var ll = self.left.evaluate(compressor);
|
||||
var rr = self.right.evaluate(compressor);
|
||||
if (ll !== self.left && ll || rr !== self.right && rr) {
|
||||
compressor.warn("Boolean || always true [{file}:{line},{col}]", self.start);
|
||||
return make_node(AST_Seq, self, {
|
||||
car: self.left,
|
||||
cdr: make_node(AST_True, self)
|
||||
}).optimize(compressor);
|
||||
}
|
||||
if (!ll) {
|
||||
return self.right.optimize(compressor);
|
||||
}
|
||||
if (!rr) {
|
||||
return self.left.optimize(compressor);
|
||||
}
|
||||
break;
|
||||
case "+":
|
||||
if (compressor.option("booleans") && self.operator == "+" && compressor.in_boolean_context()) {
|
||||
var ll = self.left.evaluate(compressor);
|
||||
var rr = self.right.evaluate(compressor);
|
||||
if (ll && typeof ll == "string") {
|
||||
@@ -3333,7 +3330,6 @@ merge(Compressor.prototype, {
|
||||
cdr: make_node(AST_True, self)
|
||||
}).optimize(compressor);
|
||||
}
|
||||
break;
|
||||
}
|
||||
if (compressor.option("comparisons") && self.is_boolean()) {
|
||||
if (!(compressor.parent() instanceof AST_Binary)
|
||||
@@ -3374,24 +3370,48 @@ merge(Compressor.prototype, {
|
||||
if (compressor.option("evaluate")) {
|
||||
switch (self.operator) {
|
||||
case "&&":
|
||||
if (self.left.is_constant()) {
|
||||
if (self.left.constant_value(compressor)) {
|
||||
compressor.warn("Condition left of && always true [{file}:{line},{col}]", self.start);
|
||||
return maintain_this_binding(compressor.parent(), self, self.right);
|
||||
} else {
|
||||
compressor.warn("Condition left of && always false [{file}:{line},{col}]", self.start);
|
||||
return maintain_this_binding(compressor.parent(), self, self.left);
|
||||
var ll = self.left.evaluate(compressor);
|
||||
if (!ll) {
|
||||
compressor.warn("Condition left of && always false [{file}:{line},{col}]", self.start);
|
||||
return maintain_this_binding(compressor.parent(), self, self.left).optimize(compressor);
|
||||
} else if (ll !== self.left) {
|
||||
compressor.warn("Condition left of && always true [{file}:{line},{col}]", self.start);
|
||||
return maintain_this_binding(compressor.parent(), self, self.right).optimize(compressor);
|
||||
}
|
||||
if (compressor.option("booleans") && compressor.in_boolean_context()) {
|
||||
var rr = self.right.evaluate(compressor);
|
||||
if (!rr) {
|
||||
compressor.warn("Boolean && always false [{file}:{line},{col}]", self.start);
|
||||
return make_node(AST_Seq, self, {
|
||||
car: self.left,
|
||||
cdr: make_node(AST_False, self)
|
||||
}).optimize(compressor);
|
||||
} else if (rr !== self.right) {
|
||||
compressor.warn("Dropping side-effect-free && in boolean context [{file}:{line},{col}]", self.start);
|
||||
return self.left.optimize(compressor);
|
||||
}
|
||||
}
|
||||
break;
|
||||
case "||":
|
||||
if (self.left.is_constant()) {
|
||||
if (self.left.constant_value(compressor)) {
|
||||
compressor.warn("Condition left of || always true [{file}:{line},{col}]", self.start);
|
||||
return maintain_this_binding(compressor.parent(), self, self.left);
|
||||
} else {
|
||||
compressor.warn("Condition left of || always false [{file}:{line},{col}]", self.start);
|
||||
return maintain_this_binding(compressor.parent(), self, self.right);
|
||||
var ll = self.left.evaluate(compressor);
|
||||
if (!ll) {
|
||||
compressor.warn("Condition left of || always false [{file}:{line},{col}]", self.start);
|
||||
return maintain_this_binding(compressor.parent(), self, self.right).optimize(compressor);
|
||||
} else if (ll !== self.left) {
|
||||
compressor.warn("Condition left of || always true [{file}:{line},{col}]", self.start);
|
||||
return maintain_this_binding(compressor.parent(), self, self.left).optimize(compressor);
|
||||
}
|
||||
if (compressor.option("booleans") && compressor.in_boolean_context()) {
|
||||
var rr = self.right.evaluate(compressor);
|
||||
if (!rr) {
|
||||
compressor.warn("Dropping side-effect-free || in boolean context [{file}:{line},{col}]", self.start);
|
||||
return self.left.optimize(compressor);
|
||||
} else if (rr !== self.right) {
|
||||
compressor.warn("Boolean || always true [{file}:{line},{col}]", self.start);
|
||||
return make_node(AST_Seq, self, {
|
||||
car: self.left,
|
||||
cdr: make_node(AST_True, self)
|
||||
}).optimize(compressor);
|
||||
}
|
||||
}
|
||||
break;
|
||||
@@ -3617,9 +3637,9 @@ merge(Compressor.prototype, {
|
||||
case "undefined":
|
||||
return make_node(AST_Undefined, self).optimize(compressor);
|
||||
case "NaN":
|
||||
return make_node(AST_NaN, self);
|
||||
return make_node(AST_NaN, self).optimize(compressor);
|
||||
case "Infinity":
|
||||
return make_node(AST_Infinity, self);
|
||||
return make_node(AST_Infinity, self).optimize(compressor);
|
||||
}
|
||||
}
|
||||
if (compressor.option("evaluate") && compressor.option("reduce_vars")) {
|
||||
@@ -3649,19 +3669,48 @@ merge(Compressor.prototype, {
|
||||
|
||||
OPT(AST_Undefined, function(self, compressor){
|
||||
if (compressor.option("unsafe")) {
|
||||
var scope = compressor.find_parent(AST_Scope);
|
||||
var undef = scope.find_variable("undefined");
|
||||
var undef = find_variable(compressor, "undefined");
|
||||
if (undef) {
|
||||
var ref = make_node(AST_SymbolRef, self, {
|
||||
name : "undefined",
|
||||
scope : scope,
|
||||
scope : undef.scope,
|
||||
thedef : undef
|
||||
});
|
||||
ref.is_undefined = true;
|
||||
return ref;
|
||||
}
|
||||
}
|
||||
return self;
|
||||
return make_node(AST_UnaryPrefix, self, {
|
||||
operator: "void",
|
||||
expression: make_node(AST_Number, self, {
|
||||
value: 0
|
||||
})
|
||||
});
|
||||
});
|
||||
|
||||
OPT(AST_Infinity, function(self, compressor){
|
||||
var retain = compressor.option("keep_infinity") && !find_variable(compressor, "Infinity");
|
||||
return retain ? self : make_node(AST_Binary, self, {
|
||||
operator: "/",
|
||||
left: make_node(AST_Number, self, {
|
||||
value: 1
|
||||
}),
|
||||
right: make_node(AST_Number, self, {
|
||||
value: 0
|
||||
})
|
||||
});
|
||||
});
|
||||
|
||||
OPT(AST_NaN, function(self, compressor){
|
||||
return find_variable(compressor, "NaN") ? make_node(AST_Binary, self, {
|
||||
operator: "/",
|
||||
left: make_node(AST_Number, self, {
|
||||
value: 0
|
||||
}),
|
||||
right: make_node(AST_Number, self, {
|
||||
value: 0
|
||||
})
|
||||
}) : self;
|
||||
});
|
||||
|
||||
var ASSIGN_OPS = [ '+', '-', '/', '*', '%', '>>', '<<', '>>>', '|', '^', '&' ];
|
||||
@@ -3979,7 +4028,7 @@ merge(Compressor.prototype, {
|
||||
OPT(AST_RegExp, literals_in_boolean_context);
|
||||
|
||||
OPT(AST_Return, function(self, compressor){
|
||||
if (self.value && is_undefined(self.value)) {
|
||||
if (self.value && is_undefined(self.value, compressor)) {
|
||||
self.value = null;
|
||||
}
|
||||
return self;
|
||||
@@ -4000,7 +4049,7 @@ merge(Compressor.prototype, {
|
||||
});
|
||||
|
||||
OPT(AST_Yield, function(self, compressor){
|
||||
if (!self.is_star && self.expression instanceof AST_Undefined) {
|
||||
if (self.expression && !self.is_star && is_undefined(self.expression, compressor)) {
|
||||
self.expression = null;
|
||||
}
|
||||
return self;
|
||||
|
||||
Reference in New Issue
Block a user