[llvm] r222922 - Add an assert and use a range loop. NFC.

Rafael Espindola rafael.espindola at gmail.com
Fri Nov 28 08:26:15 PST 2014


Author: rafael
Date: Fri Nov 28 10:26:14 2014
New Revision: 222922

URL: http://llvm.org/viewvc/llvm-project?rev=222922&view=rev
Log:
Add an assert and use a range loop. NFC.

Modified:
    llvm/trunk/lib/Linker/LinkModules.cpp

Modified: llvm/trunk/lib/Linker/LinkModules.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Linker/LinkModules.cpp?rev=222922&r1=222921&r2=222922&view=diff
==============================================================================
--- llvm/trunk/lib/Linker/LinkModules.cpp (original)
+++ llvm/trunk/lib/Linker/LinkModules.cpp Fri Nov 28 10:26:14 2014
@@ -94,13 +94,15 @@ private:
 }
 
 void TypeMapTy::addTypeMapping(Type *DstTy, Type *SrcTy) {
+  assert(SpeculativeTypes.empty());
+
   // Check to see if these types are recursively isomorphic and establish a
   // mapping between them if so.
   if (!areTypesIsomorphic(DstTy, SrcTy)) {
     // Oops, they aren't isomorphic.  Just discard this request by rolling out
     // any speculative mappings we've established.
-    for (unsigned i = 0, e = SpeculativeTypes.size(); i != e; ++i)
-      MappedTypes.erase(SpeculativeTypes[i]);
+    for (Type *Ty : SpeculativeTypes)
+      MappedTypes.erase(Ty);
   }
   SpeculativeTypes.clear();
 }





More information about the llvm-commits mailing list