[PATCH] Move the calling of emitTargetMD() later.

Rafael Ávila de Espíndola rafael.espindola at gmail.com
Wed Jun 18 09:14:34 PDT 2014


Alp Toker changed how we handled mangled names last, he is probably the correct person to review this.

================
Comment at: lib/CodeGen/CodeGenModule.cpp:3250
@@ -3252,4 +3249,3 @@
 
-  for (llvm::DenseMap<const Decl*, llvm::Value*>::iterator
-         I = LocalDeclMap.begin(), E = LocalDeclMap.end(); I != E; ++I) {
+  for (auto I = LocalDeclMap.begin(), E = LocalDeclMap.end(); I != E; ++I) {
     const Decl *D = I->first;
----------------
Can this use a range loop. If so, please commit that as a first independent improvement.

http://reviews.llvm.org/D4176






More information about the cfe-commits mailing list