Chromium Code Reviews
Descriptioncps-ir: Merge variables based on the source variable's name.
Previously we would merge based on the element, but that is a poor
strategy when multiple locals have the same name. E.g:
for (int i = 0; i < X; ++i) {...}
for (int i = 0; i < Y; ++i) {...}
The two `i` variables should obviously be merged, but they have
different elements.
BUG=
R=kmillikin@google.com
Committed: https://code.google.com/p/dart/source/detail?r=45134
Patch Set 1 #Patch Set 2 : Do not substitute parameters #Patch Set 3 : Bugfix: local variable could not actually merge with parameter #Messages
Total messages: 6 (1 generated)
|