LGTM
http://codereview.chromium.org/6873075/diff/1/src/lithium-allocator.cc File src/lithium-allocator.cc (right): http://codereview.chromium.org/6873075/diff/1/src/lithium-allocator.cc#newcode1126 src/lithium-allocator.cc:1126: LGap* gap = NULL; you can duplicate&move this variable into branches. http://codereview.chromium.org/6873075/diff/1/src/lithium-allocator.cc#newcode1128 src/lithium-allocator.cc:1128: gap = GapAt(block->first_instruction_index() + 1); GetFirstGap()? http://codereview.chromium.org/6873075/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
