[llvm] r209194 - llvm-readobj: use range-based for loop

Saleem Abdulrasool compnerd at compnerd.org
Mon May 19 22:18:06 PDT 2014


Author: compnerd
Date: Tue May 20 00:18:06 2014
New Revision: 209194

URL: http://llvm.org/viewvc/llvm-project?rev=209194&view=rev
Log:
llvm-readobj: use range-based for loop

Convert an additional site to a range based for loop.  NFC.

Modified:
    llvm/trunk/tools/llvm-readobj/COFFDumper.cpp

Modified: llvm/trunk/tools/llvm-readobj/COFFDumper.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-readobj/COFFDumper.cpp?rev=209194&r1=209193&r2=209194&view=diff
==============================================================================
--- llvm/trunk/tools/llvm-readobj/COFFDumper.cpp (original)
+++ llvm/trunk/tools/llvm-readobj/COFFDumper.cpp Tue May 20 00:18:06 2014
@@ -196,15 +196,13 @@ static error_code resolveSectionAndAddre
 // the function returns the symbol used for the relocation at the offset.
 static error_code resolveSymbol(const std::vector<RelocationRef> &Rels,
                                 uint64_t Offset, SymbolRef &Sym) {
-  for (std::vector<RelocationRef>::const_iterator RelI = Rels.begin(),
-                                                  RelE = Rels.end();
-                                                  RelI != RelE; ++RelI) {
+  for (const auto &Relocation : Rels) {
     uint64_t Ofs;
-    if (error_code EC = RelI->getOffset(Ofs))
+    if (error_code EC = Relocation.getOffset(Ofs))
       return EC;
 
     if (Ofs == Offset) {
-      Sym = *RelI->getSymbol();
+      Sym = *Relocation.getSymbol();
       return readobj_error::success;
     }
   }





More information about the llvm-commits mailing list