You must need changes in all the other platform files as well?

http://codereview.chromium.org/385069/diff/1/7
File src/arm/assembler-arm.h (right):

http://codereview.chromium.org/385069/diff/1/7#newcode444
Line 444: class Scope BASE_EMBEDDED {
Could you add the

// Enable a specified feature within a scope.

comment.

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

http://codereview.chromium.org/385069/diff/1/10#newcode5110
Line 5110: if      (Token::MUL == operation) __ fmuld(d5, d6, d7);
Please remove extra spacing.

http://codereview.chromium.org/385069

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

Reply via email to