Lithium LLabel instruction are no longer used as gap instructions.

Lithium LLabel instruction are no longer used as gap instructions.

Instead we use the first part of the first gap after
the label to insert gap-moves for resolving control-flow
and phis.
Committed: http://code.google.com/p/v8/source/detail?r=7665

[Vyacheslav Egorov (Chromium), 2011-04-19 15:01:07]

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#newcod...
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#newcod...
src/lithium-allocator.cc:1128: gap = GapAt(block->first_instruction_index() +
1);
GetFirstGap()?

[fschneider, 2011-04-19 16:01:05]

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#newcod...
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#newcod...
src/lithium-allocator.cc:1128: gap = GapAt(block->first_instruction_index() +
1);
On 2011/04/19 15:01:08, Vyacheslav Egorov wrote:
> GetFirstGap()?

Done.

[Vyacheslav Egorov (Chromium), 2011-04-19 16:10:59]

LGTM