diff options
author | Alon Zakai <alonzakai@gmail.com> | 2014-05-17 20:53:41 -0700 |
---|---|---|
committer | Alon Zakai <alonzakai@gmail.com> | 2014-05-18 10:25:00 -0700 |
commit | 57360800512dc1b23532e1c125c8e0627ccdbcfa (patch) | |
tree | 4631ae59fff7954b3547c6478d16daf34b1d702a /tools/js-optimizer.js | |
parent | 398cded47b7e08fbb753fb7b4788da29a22195bb (diff) |
fix looptemp replacements when there is a continue with a phi of the loop variable
Diffstat (limited to 'tools/js-optimizer.js')
-rw-r--r-- | tools/js-optimizer.js | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/tools/js-optimizer.js b/tools/js-optimizer.js index fe00254a..c0096df4 100644 --- a/tools/js-optimizer.js +++ b/tools/js-optimizer.js @@ -3717,11 +3717,15 @@ function eliminate(ast, memSafe) { assert(!(temp in asmData.vars)); for (var i = firstLooperUsage; i <= lastLooperUsage; i++) { var curr = i < stats.length-1 ? stats[i] : last[1]; // on the last line, just look in the condition - traverse(curr, function(node, type) { + traverse(curr, function looperToLooptemp(node, type) { if (type === 'name') { if (node[1] === looper) { node[1] = temp; } + } else if (type === 'assign' && node[2][0] === 'name') { + // do not traverse the assignment target, phi assignments to the loop variable must remain + traverse(node[3], looperToLooptemp); + return null; } }); } |