Reviewers: Erik Corry,

Description:
Fix lint errors

tbr=erik.co...@gmail.com

Please review this at http://codereview.chromium.org/1749002/show

SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/

Affected files:
  M     src/arm/assembler-arm.h
  M     src/arm/codegen-arm.cc
  M     src/arm/full-codegen-arm.cc
  M     src/arm/macro-assembler-arm.cc


Index: src/arm/assembler-arm.h
===================================================================
--- src/arm/assembler-arm.h     (revision 4457)
+++ src/arm/assembler-arm.h     (working copy)
@@ -936,9 +936,9 @@
     }

    private:
+    Assembler* assem_;
+
     DISALLOW_IMPLICIT_CONSTRUCTORS(BlockConstPoolScope);
-
-    Assembler* assem_;
   };

   // Debugging
Index: src/arm/codegen-arm.cc
===================================================================
--- src/arm/codegen-arm.cc      (revision 4457)
+++ src/arm/codegen-arm.cc      (working copy)
@@ -359,7 +359,7 @@
     masm_->bind(&check_exit_codesize);
 #endif

-    {
+    { // NOLINT
// Make sure that the constant pool is not emitted inside of the return
       // sequence.
       Assembler::BlockConstPoolScope block_const_pool(masm_);
Index: src/arm/full-codegen-arm.cc
===================================================================
--- src/arm/full-codegen-arm.cc (revision 4457)
+++ src/arm/full-codegen-arm.cc (working copy)
@@ -200,7 +200,7 @@
     masm_->bind(&check_exit_codesize);
 #endif

-    {
+    { // NOLINT
// Make sure that the constant pool is not emitted inside of the return
       // sequence.
       Assembler::BlockConstPoolScope block_const_pool(masm_);
Index: src/arm/macro-assembler-arm.cc
===================================================================
--- src/arm/macro-assembler-arm.cc      (revision 4457)
+++ src/arm/macro-assembler-arm.cc      (working copy)
@@ -117,7 +117,7 @@
   //  ldr ip, [pc, #...]
   //  blx ip

-  {
+  { // NOLINT
     // The two instructions (ldr and blx) could be separated by a constant
     // pool and the code would still work. The issue comes from the
     // patching code which expect the ldr to be just above the blx.


--
v8-dev mailing list
v8-dev@googlegroups.com
http://groups.google.com/group/v8-dev

Reply via email to