restore inline functionality disabled by #4204 (#4209)

This commit is contained in:
Alex Lam S.L
2020-10-13 02:33:49 +01:00
committed by GitHub
parent 176c09c6a5
commit 3096f6fdad

View File

@@ -7069,7 +7069,7 @@ merge(Compressor.prototype, {
return node; return node;
} }
} }
var scope, in_loop, level = -1; var child, in_loop, scope;
if (replacing && can_inject_symbols()) { if (replacing && can_inject_symbols()) {
fn._squeezed = true; fn._squeezed = true;
if (exp !== fn) fn.parent_scope = exp.scope; if (exp !== fn) fn.parent_scope = exp.scope;
@@ -7239,14 +7239,14 @@ merge(Compressor.prototype, {
function can_inject_symbols() { function can_inject_symbols() {
var defined = Object.create(null); var defined = Object.create(null);
var child; var level = 0;
scope = compressor.self(); scope = compressor.self();
do { while (!(scope instanceof AST_Scope)) {
child = scope;
scope = compressor.parent(++level);
if (scope.variables) scope.variables.each(function(def) { if (scope.variables) scope.variables.each(function(def) {
defined[def.name] = true; defined[def.name] = true;
}); });
child = scope;
scope = compressor.parent(level++);
if (scope instanceof AST_DWLoop) { if (scope instanceof AST_DWLoop) {
in_loop = []; in_loop = [];
} else if (scope instanceof AST_For) { } else if (scope instanceof AST_For) {
@@ -7259,7 +7259,7 @@ merge(Compressor.prototype, {
} else if (scope instanceof AST_SymbolRef) { } else if (scope instanceof AST_SymbolRef) {
if (scope.fixed_value() instanceof AST_Scope) return false; if (scope.fixed_value() instanceof AST_Scope) return false;
} }
} while (!(scope instanceof AST_Scope)); }
var safe_to_inject = (!(scope instanceof AST_Toplevel) || compressor.toplevel.vars) var safe_to_inject = (!(scope instanceof AST_Toplevel) || compressor.toplevel.vars)
&& (exp !== fn || fn.parent_scope.resolve() === compressor.find_parent(AST_Scope)); && (exp !== fn || fn.parent_scope.resolve() === compressor.find_parent(AST_Scope));
var inline = compressor.option("inline"); var inline = compressor.option("inline");
@@ -7362,7 +7362,7 @@ merge(Compressor.prototype, {
return true; return true;
} }
}); });
args.unshift(scope.body.indexOf(compressor.parent(level - 1)) + 1, 0); args.unshift(scope.body.indexOf(child) + 1, 0);
if (decls.length) args.push(make_node(AST_Var, fn, { if (decls.length) args.push(make_node(AST_Var, fn, {
definitions: decls definitions: decls
})); }));