LGTM. I'll take care of the last identation nit when landing. Thanks!
http://codereview.chromium.org/6274009/diff/40002/src/arm/assembler-arm.cc
File src/arm/assembler-arm.cc (right):
http://codereview.chromium.org/6274009/diff/40002/src/arm/assembler-arm.cc#newcode1470
src/arm/assembler-arm.cc
http://codereview.chromium.org/6274009/
--
v8-dev mailing list
v8-dev@googlegroups.com
http://groups.google.com/group/v8-dev
http://codereview.chromium.org/6274009/diff/21001/src/arm/constants-arm.h
File src/arm/constants-arm.h (right):
http://codereview.chromium.org/6274009/diff/21001/src/arm/constants-arm.h#newcode364
src/arm/constants-arm.h:364: call_rt_redirected = 0x10,
Fixed.
I missed this one as I nearly missed
Just saw a last comment on a few words' case.
I will fix it.
http://codereview.chromium.org/6274009/diff/21001/src/arm/assembler-arm.cc
File src/arm/assembler-arm.cc (right):
http://codereview.chromium.org/6274009/diff/21001/src/arm/assembler-arm.cc#newcode432
src/arm/assembler-arm.cc:432: retu
http://codereview.chromium.org/6274009/
--
v8-dev mailing list
v8-dev@googlegroups.com
http://groups.google.com/group/v8-dev
This looks great!
A couple of indentation changes and suggestions below. Once they are
addressed I
think we are ready to land this.
We are very inconsistent in our enumerator names. I have put in a comment
on the
things that I think we should fix before landing this patch. The rest we can
Thanks for the review!
Alexandre
http://codereview.chromium.org/6274009/diff/1/src/arm/assembler-arm.h
File src/arm/assembler-arm.h (right):
http://codereview.chromium.org/6274009/diff/1/src/arm/assembler-arm.h#newcode43
src/arm/assembler-arm.h:43: #include "constants-arm.h"
On 2011/01/20 12:4
http://codereview.chromium.org/6274009/
--
v8-dev mailing list
v8-dev@googlegroups.com
http://groups.google.com/group/v8-dev
I think we should take this opportunity to get rid of all the assembler::
namespaces in the system and just use v8 and v8::internal.
This will be a very nice cleanup. :)
http://codereview.chromium.org/6274009/diff/1/src/arm/constants-arm.h
File src/arm/constants-arm.h (right):
http://coderevie
First overall comment: no using directives. :)
I'll have a look at the actual contents soon.
http://codereview.chromium.org/6274009/diff/1/src/arm/assembler-arm.h
File src/arm/assembler-arm.h (right):
http://codereview.chromium.org/6274009/diff/1/src/arm/assembler-arm.h#newcode43
src/arm/assem
10 matches
Mail list logo