I choose the name StackOperandForArgument as both the "rsp + disp" and "rsp +
index * scale + disp" in this CL are used to access argument. Receiver is
considered as argument 0. For non-argument access, I used kPCOnStackSize instead
of kPointerSize and explained them in place.


https://codereview.chromium.org/21123008/diff/1/src/x64/builtins-x64.cc
File src/x64/builtins-x64.cc (right):

https://codereview.chromium.org/21123008/diff/1/src/x64/builtins-x64.cc#newcode711
src/x64/builtins-x64.cc:711: __ SmiToInteger32(r10, Operand(rsp,
kPCOnStackSize));
Use kPCOnStackSize instead of kPointerSize.

https://codereview.chromium.org/21123008/diff/1/src/x64/builtins-x64.cc#newcode720
src/x64/builtins-x64.cc:720: __ movq(rax, Operand(rsp, kPCOnStackSize +
kPointerSize));
Use kPCOnStackSize instead of kPointerSize.

https://codereview.chromium.org/21123008/diff/1/src/x64/stub-cache-x64.cc
File src/x64/stub-cache-x64.cc (right):

https://codereview.chromium.org/21123008/diff/1/src/x64/stub-cache-x64.cc#newcode1077
src/x64/stub-cache-x64.cc:1077: __ movq(Operand(rsp, kPCOnStackSize),
object_reg);
Use kPCOnStackSize instead of kPointerSize.

https://codereview.chromium.org/21123008/diff/1/src/x64/stub-cache-x64.cc#newcode1137
src/x64/stub-cache-x64.cc:1137: __ movq(Operand(rsp, kPCOnStackSize),
reg);
Use kPCOnStackSize instead of kPointerSize.

https://codereview.chromium.org/21123008/

--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to