[PATCH] D35116: [ARM] Tidy up ARMBaseRegisterInfo implementation

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 7 04:17:05 PDT 2017


fhahn accepted this revision.
fhahn added a comment.
This revision is now accepted and ready to land.

Looks like a NFC, maybe address the style fix before committing.



================
Comment at: lib/Target/ARM/ARMBaseRegisterInfo.cpp:662
+  for (; !MI->getOperand(i).isFI(); ++i)
+    assert(i+1 < MI->getNumOperands() &&"Instr doesn't have FrameIndex operand!");
 
----------------
I think there should be a space after &&


https://reviews.llvm.org/D35116





More information about the llvm-commits mailing list