Fix some style issues in the ARM code....

src/arm/assembler-arm.h

 // Copyright (c) 1994-2006 Sun Microsystems Inc.
 // All Rights Reserved.
 //
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions
 // are met:
 //
 // - Redistributions of source code must retain the above copyright notice,
 // this list of conditions and the following disclaimer.
 //
@@ skipping 417 matching lines @@
 class CpuFeatures : public AllStatic {
  public:
   enum Feature { VFP3 = 1 };
   // Detect features of the target CPU. Set safe defaults if the serializer
   // is enabled (snapshots must be portable).
   static void Probe();
   // Check whether a feature is supported by the target CPU.
   static bool IsSupported(Feature f) {
     if (f == VFP3 && !FLAG_enable_vfp3) return false;
 
-    return (supported_ & (static_cast<uint64_t>(1) << f)) != 0;
+    return (supported_ & (1u << f)) != 0;
   }
   // Check whether a feature is currently enabled.
   static bool IsEnabled(Feature f) {
-    return (enabled_ & (static_cast<uint64_t>(1) << f)) != 0;
+    return (enabled_ & (1u << f)) != 0;
   }
+  class Scope BASE_EMBEDDED {

[Mads Ager (chromium), 2009/11/12 13:25:24]

Could you add the

// Enable a specified feature within a scope.

comment.

[Erik Corry, 2009/11/12 13:42:37]

Done.

+#ifdef DEBUG
+   public:
+    explicit Scope(Feature f) {
+      ASSERT(CpuFeatures::IsSupported(f));
+      old_enabled_ = CpuFeatures::enabled_;
+      CpuFeatures::enabled_ |= 1u << f;
+    }
+    ~Scope() { CpuFeatures::enabled_ = old_enabled_; }
+   private:
+    unsigned old_enabled_;
+#else
+   public:
+    explicit Scope(Feature f) {}
+#endif
+  };
 
  private:
-  static uint64_t supported_;
-  static uint64_t enabled_;
+  static unsigned supported_;
+  static unsigned enabled_;
 };
 
 
 typedef int32_t Instr;
 
 
 extern const Instr kMovLrPc;
 extern const Instr kLdrPCPattern;
 
 
@@ skipping 510 matching lines @@
   void RecordRelocInfo(RelocInfo::Mode rmode, intptr_t data = 0);
 
   friend class RegExpMacroAssemblerARM;
   friend class RelocInfo;
   friend class CodePatcher;
 };
 
 } }  // namespace v8::internal
 
 #endif  // V8_ARM_ASSEMBLER_ARM_H_