http://codereview.chromium.org/7024047/diff/8001/src/arm/code-stubs-arm.cc File src/arm/code-stubs-arm.cc (right):
http://codereview.chromium.org/7024047/diff/8001/src/arm/code-stubs-arm.cc#newcode3975 src/arm/code-stubs-arm.cc:3975: // r1 = parameter count (untagged) Should be done now. http://codereview.chromium.org/7024047/diff/8001/src/arm/code-stubs-arm.cc#newcode4004 src/arm/code-stubs-arm.cc:4004: __ mov(r9, r1); On 2011/06/10 11:51:52, Kevin Millikin wrote:
Not a big deal, but you could save the mov(r1, r9) below by using (and destroying) r9 instead of r1.
Done. http://codereview.chromium.org/7024047/diff/8001/src/x64/code-stubs-x64.cc File src/x64/code-stubs-x64.cc (right): http://codereview.chromium.org/7024047/diff/8001/src/x64/code-stubs-x64.cc#newcode2006 src/x64/code-stubs-x64.cc:2006: __ push(rbx); I have removed the push instructions, the code is now more similar to the ARM version. http://codereview.chromium.org/7024047/diff/8001/src/x64/code-stubs-x64.cc#newcode2091 src/x64/code-stubs-x64.cc:2091: __ push(rax); On 2011/06/10 11:51:52, Kevin Millikin wrote:
May be able to avoid this push.
Done. http://codereview.chromium.org/7024047/diff/8001/src/x64/code-stubs-x64.cc#newcode2116 src/x64/code-stubs-x64.cc:2116: __ push(rcx); On 2011/06/10 11:51:52, Kevin Millikin wrote:
Also here.
Done. http://codereview.chromium.org/7024047/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
