diff options
Diffstat (limited to 'src')
| -rw-r--r-- | src/jsifier.js | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/src/jsifier.js b/src/jsifier.js index 115a77d4..452b2d9e 100644 --- a/src/jsifier.js +++ b/src/jsifier.js @@ -1132,13 +1132,16 @@ function JSify(data, functionsOnly, givenFunctions) { }); makeFuncLineActor('switch', function(item) { // use a switch if the range is not too big or sparse - var maxx = 0; + var minn = Infinity, maxx = -Infinity; item.switchLabels.forEach(function(switchLabel) { - maxx = Math.max(maxx, Math.abs(parseInt(switchLabel.value))); + var curr = Math.abs(parseInt(switchLabel.value)); + minn = Math.min(minn, curr); + maxx = Math.max(maxx, curr); }); - var useIfs = (item.switchLabels.length+1) < 6 || maxx > 2*1024*1024 || (maxx/item.switchLabels.length) > 10*1024; // heuristics + var range = maxx - minn; + var useIfs = (item.switchLabels.length+1) < 6 || range > 2*1024*1024 || (range/item.switchLabels.length) > 10*1024; // heuristics if (VERBOSE && useIfs && item.switchLabels.length > 2) { - warn('not optimizing llvm switch into js switch because ' + [maxx, maxx/item.switchLabels.length]); + warn('not optimizing llvm switch into js switch because ' + [range, range/item.switchLabels.length]); } var phiSets = calcPhiSets(item); |