https://chromiumcodereview.appspot.com/9365057/diff/12001/src/hydrogen.cc
File src/hydrogen.cc (right):

https://chromiumcodereview.appspot.com/9365057/diff/12001/src/hydrogen.cc#newcode1179
src/hydrogen.cc:1179: if
(value->gvn_flags().ContainsAnyOf(depends_flags)) {
You'd also need to handle TransitionElementsKind specially here.

Generally, I think it would be cleaner to treat them specially when
doing loop code motion, rather than here by removing all depends-flags
from TransitionElementsKind.

There is already special code in ProcessLoopBlock for transitions, and
this way you can avoid having code for special treatment in two
different places.

https://chromiumcodereview.appspot.com/9365057/

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to