Bug 1498618 - Never apply clang-format on code in {{{ check_macroassembler_style r=nbp

Differential Revision: https://phabricator.services.mozilla.com/D8571

--HG--
extra : moz-landing-system : lando

js/src/jit/MacroAssembler-inl.h

@@ -33,6 +33,7 @@
 namespace js {
 namespace jit {
 
+// clang-format off
 //{{{ check_macroassembler_style
 // ===============================================================
 // Stack manipulation functions.
@@ -799,6 +800,7 @@ template void MacroAssembler::storeFloat32(FloatRegister src, const Address& des
 template void MacroAssembler::storeFloat32(FloatRegister src, const BaseIndex& dest);
 
 //}}} check_macroassembler_style
+// clang-format on
 // ===============================================================
 
 #ifndef JS_CODEGEN_ARM64

js/src/jit/MacroAssembler.cpp

@@ -2973,6 +2973,7 @@ MacroAssembler::subFromStackPtr(Register reg)
 }
 #endif // JS_CODEGEN_ARM64
 
+// clang-format off
 //{{{ check_macroassembler_style
 // ===============================================================
 // Stack manipulation functions.
@@ -3816,6 +3817,7 @@ MacroAssembler::boundsCheck32PowerOfTwo(Register index, uint32_t length, Label*
 }
 
 //}}} check_macroassembler_style
+// clang-format on
 
 void
 MacroAssembler::memoryBarrierBefore(const Synchronization& sync) {

js/src/jit/MacroAssembler.h

@@ -350,6 +350,7 @@ class MacroAssembler : public MacroAssemblerSpecific
     void Push(RegisterOrSP reg);
 #endif
 
+    // clang-format off
     //{{{ check_macroassembler_decl_style
   public:
     // ===============================================================
@@ -2056,6 +2057,7 @@ class MacroAssembler : public MacroAssemblerSpecific
     void speculationBarrier() PER_SHARED_ARCH;
 
     //}}} check_macroassembler_decl_style
+    // clang-format on
   public:
 
     // Emits a test of a value against all types in a TypeSet. A scratch
@@ -2819,6 +2821,7 @@ class IonHeapMacroAssembler : public MacroAssembler
     }
 };
 
+// clang-format off
 //{{{ check_macroassembler_style
 inline uint32_t
 MacroAssembler::framePushed() const
@@ -2847,6 +2850,7 @@ MacroAssembler::implicitPop(uint32_t bytes)
     adjustFrame(-int32_t(bytes));
 }
 //}}} check_macroassembler_style
+// clang-format on
 
 static inline Assembler::DoubleCondition
 JSOpToDoubleCondition(JSOp op)

js/src/jit/arm/MacroAssembler-arm-inl.h

@@ -12,6 +12,7 @@
 namespace js {
 namespace jit {
 
+// clang-format off
 //{{{ check_macroassembler_style
 
 void
@@ -2334,6 +2335,7 @@ MacroAssembler::clampIntToUint8(Register reg)
 }
 
 //}}} check_macroassembler_style
+// clang-format on
 // ===============================================================
 
 void

js/src/jit/arm/MacroAssembler-arm.cpp

@@ -4362,6 +4362,7 @@ MacroAssembler::subFromStackPtr(Imm32 imm32)
     }
 }
 
+// clang-format off
 //{{{ check_macroassembler_style
 // ===============================================================
 // MacroAssembler high-level usage.
@@ -6175,6 +6176,7 @@ MacroAssembler::speculationBarrier()
 }
 
 //}}} check_macroassembler_style
+// clang-format on
 
 void
 MacroAssemblerARM::wasmTruncateToInt32(FloatRegister input, Register output, MIRType fromType,

js/src/jit/arm64/MacroAssembler-arm64-inl.h

@@ -12,6 +12,7 @@
 namespace js {
 namespace jit {
 
+// clang-format off
 //{{{ check_macroassembler_style
 
 void
@@ -1981,6 +1982,7 @@ MacroAssembler::clampIntToUint8(Register reg)
 }
 
 //}}} check_macroassembler_style
+// clang-format on
 // ===============================================================
 
 void

js/src/jit/arm64/MacroAssembler-arm64.cpp

@@ -413,6 +413,7 @@ MacroAssembler::Push(RegisterOrSP reg)
     adjustFrame(sizeof(intptr_t));
 }
 
+// clang-format off
 //{{{ check_macroassembler_style
 // ===============================================================
 // MacroAssembler high-level usage.
@@ -2024,6 +2025,7 @@ MacroAssembler::speculationBarrier()
 }
 
 //}}} check_macroassembler_style
+// clang-format off
 
 } // namespace jit
 } // namespace js

js/src/jit/mips-shared/MacroAssembler-mips-shared-inl.h

@@ -12,6 +12,7 @@
 namespace js {
 namespace jit {
 
+// clang-format off
 //{{{ check_macroassembler_style
 
 void
@@ -1110,6 +1111,7 @@ MacroAssembler::clampIntToUint8(Register reg)
 }
 
 //}}} check_macroassembler_style
+// clang-format on
 // ===============================================================
 
 } // namespace jit

js/src/jit/mips-shared/MacroAssembler-mips-shared.cpp

@@ -1439,6 +1439,7 @@ MacroAssemblerMIPSShared::asMasm() const
     return *static_cast<const MacroAssembler*>(this);
 }
 
+// clang-format off
 //{{{ check_macroassembler_style
 // ===============================================================
 // MacroAssembler high-level usage.
@@ -2927,3 +2928,5 @@ MacroAssembler::speculationBarrier()
 {
     MOZ_CRASH();
 }
+//}}} check_macroassembler_style
+// clang-format on

js/src/jit/mips32/MacroAssembler-mips32-inl.h

@@ -14,6 +14,7 @@
 namespace js {
 namespace jit {
 
+// clang-format off
 //{{{ check_macroassembler_style
 
 void
@@ -1011,6 +1012,7 @@ MacroAssembler::branchTruncateFloat32MaybeModUint32(FloatRegister src, Register
 }
 
 //}}} check_macroassembler_style
+// clang-format on
 // ===============================================================
 
 void

js/src/jit/mips32/MacroAssembler-mips32.cpp

@@ -2109,6 +2109,7 @@ MacroAssembler::subFromStackPtr(Imm32 imm32)
     }
 }
 
+// clang-format on
 //{{{ check_macroassembler_style
 // ===============================================================
 // Stack manipulation functions.
@@ -2967,3 +2968,4 @@ MacroAssembler::convertUInt64ToDouble(Register64 src, FloatRegister dest, Regist
 }
 
 //}}} check_macroassembler_style
+// clang-format on

js/src/jit/mips64/MacroAssembler-mips64-inl.h

@@ -14,6 +14,7 @@
 namespace js {
 namespace jit {
 
+// clang-format off
 //{{{ check_macroassembler_style
 
 void
@@ -758,6 +759,7 @@ MacroAssembler::branchTruncateFloat32MaybeModUint32(FloatRegister src, Register
 }
 
 //}}} check_macroassembler_style
+// clang-format on
 // ===============================================================
 
 // The specializations for cmpPtrSet are outside the braces because check_macroassembler_style can't yet

js/src/jit/mips64/MacroAssembler-mips64.cpp

@@ -1948,6 +1948,7 @@ MacroAssembler::subFromStackPtr(Imm32 imm32)
     }
 }
 
+// clang-format off
 //{{{ check_macroassembler_style
 // ===============================================================
 // Stack manipulation functions.
@@ -2759,3 +2760,4 @@ MacroAssembler::convertUInt64ToFloat32(Register64 src_, FloatRegister dest, Regi
 }
 
 //}}} check_macroassembler_style
+// clang-format on

js/src/jit/x64/MacroAssembler-x64-inl.h

@@ -14,6 +14,7 @@
 namespace js {
 namespace jit {
 
+// clang-format off
 //{{{ check_macroassembler_style
 // ===============================================================
 
@@ -964,6 +965,7 @@ MacroAssembler::truncateDoubleToUInt64(Address src, Address dest, Register temp,
 }
 
 //}}} check_macroassembler_style
+// clang-format on
 // ===============================================================
 
 void

js/src/jit/x64/MacroAssembler-x64.cpp

@@ -303,6 +303,7 @@ MacroAssembler::subFromStackPtr(Imm32 imm32)
     }
 }
 
+// clang-format off
 //{{{ check_macroassembler_style
 // ===============================================================
 // ABI function calls.
@@ -1038,3 +1039,4 @@ MacroAssembler::wasmAtomicEffectOp64(const wasm::MemoryAccessDesc& access, Atomi
 }
 
 //}}} check_macroassembler_style
+// clang-format on

js/src/jit/x86-shared/MacroAssembler-x86-shared-inl.h

@@ -12,6 +12,7 @@
 namespace js {
 namespace jit {
 
+// clang-format off
 //{{{ check_macroassembler_style
 // ===============================================================
 // Move instructions
@@ -1331,6 +1332,7 @@ MacroAssembler::clampIntToUint8(Register reg)
 }
 
 //}}} check_macroassembler_style
+// clang-format on
 // ===============================================================
 
 } // namespace jit

js/src/jit/x86-shared/MacroAssembler-x86-shared.cpp

@@ -270,6 +270,7 @@ MacroAssemblerX86Shared::minMaxFloat32(FloatRegister first, FloatRegister second
     bind(&done);
 }
 
+// clang-format off
 //{{{ check_macroassembler_style
 // ===============================================================
 // MacroAssembler high-level usage.
@@ -1673,3 +1674,4 @@ MacroAssembler::speculationBarrier()
 }
 
 //}}} check_macroassembler_style
+// clang-format on

js/src/jit/x86/MacroAssembler-x86-inl.h

@@ -14,6 +14,7 @@
 namespace js {
 namespace jit {
 
+// clang-format off
 //{{{ check_macroassembler_style
 
 void
@@ -1182,6 +1183,7 @@ MacroAssembler::truncateDoubleToUInt64(Address src, Address dest, Register temp,
 }
 
 //}}} check_macroassembler_style
+// clang-format on
 // ===============================================================
 
 // Note: this function clobbers the source register.

js/src/jit/x86/MacroAssembler-x86.cpp

@@ -302,6 +302,7 @@ MacroAssembler::subFromStackPtr(Imm32 imm32)
     }
 }
 
+// clang-format off
 //{{{ check_macroassembler_style
 // ===============================================================
 // ABI function calls.
@@ -1274,4 +1275,5 @@ MacroAssembler::convertInt64ToFloat32(Register64 input, FloatRegister output)
 }
 
 //}}} check_macroassembler_style
+// clang-format on