LGTM
http://codereview.chromium.org/6312124/diff/1/src/x64/lithium-codegen-x64.cc File src/x64/lithium-codegen-x64.cc (right): http://codereview.chromium.org/6312124/diff/1/src/x64/lithium-codegen-x64.cc#newcode1724 src/x64/lithium-codegen-x64.cc:1724: __ cmpq(ToRegister(instr->index()), ToOperand(instr->length())); Maybe we should create a macro, TO_REGISTER_OR_OPERAND, we are going to do this a lot! I am fine with it not being in this change though. http://codereview.chromium.org/6312124/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
