majnemer added a subscriber: majnemer.

================
Comment at: lib/CodeGen/CGVTables.cpp:190
@@ -189,3 +189,3 @@
     // Fix up the returned value, if necessary.
-    for (llvm::Function::iterator I = Fn->begin(), E = Fn->end(); I != E; I++) 
{
-      llvm::Instruction *T = I->getTerminator();
+    for (auto &BasicBlock : *Fn) {
+      llvm::Instruction *T = BasicBlock.getTerminator();
----------------
I'd name the variable `BB` instead because `BasicBlock` is the name of a class 
in LLVM.


http://reviews.llvm.org/D11446




_______________________________________________
cfe-commits mailing list
cfe-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to