LGTM

http://codereview.chromium.org/3119005/diff/7001/8001
File src/arm/codegen-arm.cc (right):

http://codereview.chromium.org/3119005/diff/7001/8001#newcode4693
src/arm/codegen-arm.cc:4693: Register result = r6;
Please add a comment that this register is used without notifying the
virtual frame (OK at the moment because the virtual frame doesn't use r6
yet).

http://codereview.chromium.org/3119005/diff/7001/8001#newcode5129
src/arm/codegen-arm.cc:5129: Register result = r0;  // Free as frame jas
just been spilled.
jas -> has

http://codereview.chromium.org/3119005/show

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

Reply via email to