LGTM with an idea for further simplification.

http://codereview.chromium.org/10868106/diff/4001/src/lithium.cc
File src/lithium.cc (right):

http://codereview.chromium.org/10868106/diff/4001/src/lithium.cc#newcode393
src/lithium.cc:393: LChunk* LChunk::NewChunk(HGraph* graph,
CompilationInfo* info) {
Instead of passing in |info|, can't you just use graph->info() (as line
403 does anyway)?

http://codereview.chromium.org/10868106/

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

Reply via email to