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

http://codereview.chromium.org/6326003/diff/1/src/x64/lithium-x64.cc#newcode318
src/x64/lithium-x64.cc:318: return LStackSlot::Create(index);
Made identical to ia32.

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

http://codereview.chromium.org/6326003/diff/3001/src/x64/lithium-codegen-x64.cc#newcode121
src/x64/lithium-codegen-x64.cc:121: #ifdef _MSC_VER
Will do it there as well.

http://codereview.chromium.org/6326003/diff/3001/src/x64/lithium-codegen-x64.cc#newcode125
src/x64/lithium-codegen-x64.cc:125: for (int offset = slots *
kPointerSize - 4 * KB;
Sadly not, but I'll define a local constant.
It's not something that necessarily makes sense on all platforms.

http://codereview.chromium.org/6326003/

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

Reply via email to