LGTM with nits.

https://chromiumcodereview.appspot.com/11099064/diff/1/src/arm/macro-assembler-arm.cc
File src/arm/macro-assembler-arm.cc (right):

https://chromiumcodereview.appspot.com/11099064/diff/1/src/arm/macro-assembler-arm.cc#newcode3742
src/arm/macro-assembler-arm.cc:3742: FieldMemOperand(temp,
TransitionArray::kDescriptorsOffset));
nit: fits on one line

https://chromiumcodereview.appspot.com/11099064/diff/1/src/x64/macro-assembler-x64.cc
File src/x64/macro-assembler-x64.cc (right):

https://chromiumcodereview.appspot.com/11099064/diff/1/src/x64/macro-assembler-x64.cc#newcode2940
src/x64/macro-assembler-x64.cc:2940: FieldOperand(temp,
TransitionArray::kDescriptorsOffset));
nit: fits on one line

https://chromiumcodereview.appspot.com/11099064/

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

Reply via email to