[PATCH] D35494: [ARM|CodeGen] Improve the code in FastISel

Tim Northover via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 17 11:56:51 PDT 2017


t.p.northover accepted this revision.
t.p.northover added a comment.
This revision is now accepted and ready to land.

Looks like a good change other than one nit. Feel free to commit with that fixed.



================
Comment at: lib/Target/ARM/ARMFastISel.cpp:2236-2238
+  //for (unsigned i = 0; i < I->getNumOperands(); ++i) {
+  for (Value *Op :  I->operands()) {
+    //Value *Op = I->getOperand(i);
----------------
Commented code here.


https://reviews.llvm.org/D35494





More information about the llvm-commits mailing list