Clean up emitInst() in the integrated ARM assembler.

src/IceAssemblerARM32.h

Index: src/IceAssemblerARM32.h
diff --git a/src/IceAssemblerARM32.h b/src/IceAssemblerARM32.h
index f7654b298666fea9b38965ae604b79c797195bd2..76ec3d0c809106ca4dd8af896d498f871147f582 100644
--- a/src/IceAssemblerARM32.h
+++ b/src/IceAssemblerARM32.h
@@ -455,7 +455,10 @@ private:
 
   void bindCfgNodeLabel(const CfgNode *Node) override;
 
-  void emitInst(IValueT Value) { Buffer.emit<IValueT>(Value); }
+  void emitInst(IValueT Value) {
+    AssemblerBuffer::EnsureCapacity ensured(&Buffer);

[Jim Stichnoth, 2016/01/25 16:57:14]

John has a coding style I like where these RAII placeholder variables are named
"_".  I.e.:

  AssemblerBuffer::EnsureCapacity _(&Buffer);

[Karl, 2016/01/25 17:14:07]

I really don't like this style! However, if we are using it, so will I.

+    Buffer.emit<IValueT>(Value);
+  }
 
   // List of possible checks to apply when calling emitType01() (below).
   enum EmitChecks { NoChecks, RdIsPcAndSetFlags };