[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