LGTM

http://codereview.chromium.org/8065004/diff/3002/src/arm/ic-arm.cc
File src/arm/ic-arm.cc (right):

http://codereview.chromium.org/8065004/diff/3002/src/arm/ic-arm.cc#newcode1323
src/arm/ic-arm.cc:1323: // r2: receiver.
r5: receiver map

http://codereview.chromium.org/8065004/diff/3002/src/arm/ic-arm.cc#newcode1413
src/arm/ic-arm.cc:1413: __ StoreNumberToDoubleElements(value, key,
receiver, elements,
all arguments on one line or each on seperate line (on one line will not
work here I guess :-) )

http://codereview.chromium.org/8065004/diff/3002/src/arm/macro-assembler-arm.cc
File src/arm/macro-assembler-arm.cc (right):

http://codereview.chromium.org/8065004/diff/3002/src/arm/macro-assembler-arm.cc#newcode1953
src/arm/macro-assembler-arm.cc:1953: // scratch is now effective address
of the double element
scratch1

http://codereview.chromium.org/8065004/diff/3002/src/arm/stub-cache-arm.cc
File src/arm/stub-cache-arm.cc (right):

http://codereview.chromium.org/8065004/diff/3002/src/arm/stub-cache-arm.cc#newcode4433
src/arm/stub-cache-arm.cc:4433: __
StoreNumberToDoubleElements(value_reg, key_reg, receiver_reg,
elements_reg,
all arguments on one line or each on separate lines

http://codereview.chromium.org/8065004/

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to