LGTM
http://codereview.chromium.org/555147/diff/1/3 File src/x64/codegen-x64.cc (right): http://codereview.chromium.org/555147/diff/1/3#newcode7503 src/x64/codegen-x64.cc:7503: Register left = rdx; Does it add much to rename to left and right here. The method is so short that I don't think it makes it easier to read. http://codereview.chromium.org/555147/diff/1/3#newcode7515 src/x64/codegen-x64.cc:7515: Register left = rdx; Same here. http://codereview.chromium.org/555147/diff/1/3#newcode7925 src/x64/codegen-x64.cc:7925: // edx and eax. edx -> rdx eax -> rax http://codereview.chromium.org/555147 -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
