https://chromiumcodereview.appspot.com/106723002/diff/20001/src/hydrogen-gvn.cc
File src/hydrogen-gvn.cc (right):
https://chromiumcodereview.appspot.com/106723002/diff/20001/src/hydrogen-gvn.cc#newcode409
src/hydrogen-gvn.cc:409: if (block->IsLoopHeader()) {
Can you turn this whole thing into a do {} while() ?
https://chromiumcodereview.appspot.com/106723002/diff/20001/src/hydrogen-gvn.cc#newcode573
src/hydrogen-gvn.cc:573: TRACE_GVN_3("Hoisting loop invariant
instruction %d %d %s\n",
would like to see
"Hoisting loop invariant instruction i%d to block B%d" and I think you
can drop the loop kills for now.
https://chromiumcodereview.appspot.com/106723002/diff/20001/test/mjsunit/regress/regress-context-osr.js
File test/mjsunit/regress/regress-context-osr.js (right):
https://chromiumcodereview.appspot.com/106723002/diff/20001/test/mjsunit/regress/regress-context-osr.js#newcode39
test/mjsunit/regress/regress-context-osr.js:39: }
Can you please write versions for 4 and 5 loop nesting levels? You may
also be able to reduce the OSR count as well.
https://chromiumcodereview.appspot.com/106723002/
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.
