http://codereview.chromium.org/6570005/diff/1/src/x64/lithium-codegen-x64.cc
File src/x64/lithium-codegen-x64.cc (right):

http://codereview.chromium.org/6570005/diff/1/src/x64/lithium-codegen-x64.cc#newcode3359
src/x64/lithium-codegen-x64.cc:3359: __ movq(rsi, Operand(rbp,
StandardFrameConstants::kContextOffset));
On 2011/02/23 10:03:12, William Hesse wrote:
I thought that rsi was still guaranteed to contain the context on X64,
and was
only freed up on ia32.  Why is this needed?

You are absolutely right. This is not needed. Thanks.

http://codereview.chromium.org/6570005/

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

Reply via email to