http://codereview.chromium.org/569010/diff/1/10
File src/arm/codegen-arm.cc (right):

http://codereview.chromium.org/569010/diff/1/10#newcode179
src/arm/codegen-arm.cc:179: if (strlen(FLAG_stop_at) > 0 &&
On 2010/02/03 00:02:12, Kevin Millikin wrote:
This should probably be lifted outside the if.

Done.

http://codereview.chromium.org/569010/diff/1/8
File src/fast-codegen.cc (right):

http://codereview.chromium.org/569010/diff/1/8#newcode33
src/fast-codegen.cc:33: #include "codegen.h"
On 2010/02/03 00:02:12, Kevin Millikin wrote:
I'm surprised you need both codegen.h and codegen-inl.h.

I don't need this additional include.  Removed.

http://codereview.chromium.org/569010

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

Reply via email to