http://codereview.chromium.org/7331014/diff/1/src/arm/regexp-macro-assembler-arm.h
File src/arm/regexp-macro-assembler-arm.h (right):

http://codereview.chromium.org/7331014/diff/1/src/arm/regexp-macro-assembler-arm.h#newcode32
src/arm/regexp-macro-assembler-arm.h:32: #include
"arm/assembler-arm-inl.h"
Drive-by comments:

I don't know if it's a good practice, but we usually assume that
xxx-inl.h includes xxx.h already.  That is, we usually include one or
the other, but not both.

Also, you should try to remember to the copyright dates :).

http://codereview.chromium.org/7331014/

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

Reply via email to