LGTM except for the inobject property load case.
http://codereview.chromium.org/594008/diff/1/2 File src/arm/fast-codegen-arm.cc (right): http://codereview.chromium.org/594008/diff/1/2#newcode111 src/arm/fast-codegen-arm.cc:111: __ ldr(destination(), FieldMemOperand(scratch0(), offset)); scratch0() receiver_reg()? http://codereview.chromium.org/594008/diff/1/7 File src/ia32/fast-codegen-ia32.cc (right): http://codereview.chromium.org/594008/diff/1/7#newcode115 src/ia32/fast-codegen-ia32.cc:115: __ mov(destination(), FieldOperand(scratch0(), offset)); scratch0() -> receiver_reg()? http://codereview.chromium.org/594008/diff/1/8 File src/x64/fast-codegen-x64.cc (right): http://codereview.chromium.org/594008/diff/1/8#newcode114 src/x64/fast-codegen-x64.cc:114: __ movq(destination(), FieldOperand(scratch0(), offset)); scratch0() -> receiver_reg()? http://codereview.chromium.org/594008 -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
