LGTM, with question.

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));
I thought that rsi was still guaranteed to contain the context on X64,
and was only freed up on ia32.  Why is this needed?

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

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

Reply via email to