Addressed comments and uploaded 3 missing files.
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; On 2011/04/19 15:01:08, Vyacheslav Egorov wrote:
you can duplicate&move this variable into branches.
Done. http://codereview.chromium.org/6873075/diff/1/src/lithium-allocator.cc#newcode1128 src/lithium-allocator.cc:1128: gap = GapAt(block->first_instruction_index() + 1); On 2011/04/19 15:01:08, Vyacheslav Egorov wrote:
GetFirstGap()?
Done. http://codereview.chromium.org/6873075/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
