JoBoEVD/UglifyJS
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Speedup unused compress option for already minified code

Browse Source 
Fixes: #321 #917 #1022
This commit is contained in:
kzc
2016-03-28 17:58:50 -04:00
parent 9bcf702a6e
commit 45ddb9caeb
2 changed files with 16 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
lib/compress.js
View File

@@ -1242,6 +1242,7 @@ merge(Compressor.prototype, {
&& !self.uses_eval && !self.uses_eval
) { ) {
var in_use = []; var in_use = [];
var in_use_ids = {}; // avoid expensive linear scans of in_use
var initializations = new Dictionary(); var initializations = new Dictionary();
// pass 1: find out which symbols are directly used in // pass 1: find out which symbols are directly used in
// this scope (not in nested scopes). // this scope (not in nested scopes).
@@ -1264,7 +1265,11 @@ merge(Compressor.prototype, {
return true; return true;
} }
if (node instanceof AST_SymbolRef) { if (node instanceof AST_SymbolRef) {
push_uniq(in_use, node.definition()); var node_def = node.definition();
if (!(node_def.id in in_use_ids)) {
in_use_ids[node_def.id] = true;
in_use.push(node_def);
}
return true; return true;
} }
if (node instanceof AST_Scope) { if (node instanceof AST_Scope) {
@@ -1287,7 +1292,11 @@ merge(Compressor.prototype, {
if (init) init.forEach(function(init){ if (init) init.forEach(function(init){
var tw = new TreeWalker(function(node){ var tw = new TreeWalker(function(node){
if (node instanceof AST_SymbolRef) { if (node instanceof AST_SymbolRef) {
push_uniq(in_use, node.definition()); var node_def = node.definition();
if (!(node_def.id in in_use_ids)) {
in_use_ids[node_def.id] = true;
in_use.push(node_def);
}
} }
}); });
init.walk(tw); init.walk(tw);
@@ -1315,7 +1324,7 @@ merge(Compressor.prototype, {
} }
} }
if (node instanceof AST_Defun && node !== self) { if (node instanceof AST_Defun && node !== self) {
if (!member(node.name.definition(), in_use)) { if (!(node.name.definition().id in in_use_ids)) {
compressor.warn("Dropping unused function {name} [{file}:{line},{col}]", { compressor.warn("Dropping unused function {name} [{file}:{line},{col}]", {
name : node.name.name, name : node.name.name,
file : node.name.start.file, file : node.name.start.file,
@@ -1328,7 +1337,7 @@ merge(Compressor.prototype, {
} }
if (node instanceof AST_Definitions && !(tt.parent() instanceof AST_ForIn)) { if (node instanceof AST_Definitions && !(tt.parent() instanceof AST_ForIn)) {
var def = node.definitions.filter(function(def){ var def = node.definitions.filter(function(def){
if (member(def.name.definition(), in_use)) return true; if (def.name.definition().id in in_use_ids) return true;
var w = { var w = {
name : def.name.name, name : def.name.name,
file : def.name.start.file, file : def.name.start.file,

3
lib/scope.js
View File

@@ -53,8 +53,11 @@ function SymbolDef(scope, index, orig) {
this.undeclared = false; this.undeclared = false;
this.constant = false; this.constant = false;
this.index = index; this.index = index;
this.id = SymbolDef.next_id++;
}; };
SymbolDef.next_id = 1;
SymbolDef.prototype = { SymbolDef.prototype = {
unmangleable: function(options) { unmangleable: function(options) {
if (!options) options = {}; if (!options) options = {};