http://codereview.chromium.org/3144002/diff/4002/13004 File src/arm/ic-arm.cc (right):
http://codereview.chromium.org/3144002/diff/4002/13004#newcode1143 src/arm/ic-arm.cc:1143: __ ldr(r4, FieldMemOperand(r1, JSObject::kElementsOffset)); Do we need to load elements, shouldn't they be in r2 already (otherwise comment for GenerateFastArrayLoad should be corrected). http://codereview.chromium.org/3144002/diff/4002/13029 File src/x64/ic-x64.cc (right): http://codereview.chromium.org/3144002/diff/4002/13029#newcode640 src/x64/ic-x64.cc:640: __ movq(rcx, FieldOperand(rdx, JSObject::kElementsOffset)); again looks like rcx should already hold elements. http://codereview.chromium.org/3144002/show -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
