X64 platform: Add more register allocator functions.

X64 platform: Add more register allocator functions.

Committed: http://code.google.com/p/v8/source/detail?r=2177

[William Hesse, 2009-06-15 16:32:51]

Some little changes for you.

[Lasse Reichstein, 2009-06-16 08:19:49]

LGTM

http://codereview.chromium.org/125131/diff/1/5
File src/x64/assembler-x64-inl.h (left):

http://codereview.chromium.org/125131/diff/1/5#oldcode157
Line 157: intptr_t* p = reinterpret_cast<intptr_t*>(pc_ + 1);
Why remove this? Is it not used by the debugger anyway?

[Kevin Millikin (Chromium), 2009-06-16 08:27:03]

http://codereview.chromium.org/125131/diff/19/1004
File src/x64/macro-assembler-x64.cc (right):

http://codereview.chromium.org/125131/diff/19/1004#newcode536
Line 536: // Advances rdx to the end of the Code object headers, to the start of
headers ==> header

http://codereview.chromium.org/125131/diff/19/1005
File src/x64/virtual-frame-x64.cc (right):

http://codereview.chromium.org/125131/diff/19/1005#newcode102
Line 102: // debugger. See VisitReturnStatement for the full return sequence.
The return sequence is not in VisitReturnStatement.  It's in
GenerateReturnSequence.

[William Hesse, 2009-06-16 11:45:26]

http://codereview.chromium.org/125131/diff/1/5
File src/x64/assembler-x64-inl.h (left):

http://codereview.chromium.org/125131/diff/1/5#oldcode157
Line 157: intptr_t* p = reinterpret_cast<intptr_t*>(pc_ + 1);
On 2009/06/16 08:19:50, Lasse Reichstein wrote:
> Why remove this? Is it not used by the debugger anyway?

It needs to be rewritten for the x64 platform, if needed, and it was breaking
the build of the new test.