llvm-objdump: Use range-based-for loop and fix format.
authorRui Ueyama <ruiu@google.com>
Tue, 4 Mar 2014 00:03:09 +0000 (00:03 +0000)
committerRui Ueyama <ruiu@google.com>
Tue, 4 Mar 2014 00:03:09 +0000 (00:03 +0000)
This is a small cleanup before making a bit larger change to this function.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@202770 91177308-0d34-0410-b5e6-96231b3b80d8

tools/llvm-objdump/COFFDump.cpp

index 0b7f64d39e6309ac5cc04d8be4d4db374a2f3efd..424ae7c71d5d099409d99144f25fab50a8cde899 100644 (file)
@@ -414,28 +414,31 @@ void llvm::printCOFFUnwindInfo(const COFFObjectFile *Obj) {
     ArrayRef<RuntimeFunction> RFs(
                   reinterpret_cast<const RuntimeFunction *>(Contents.data()),
                                   Contents.size() / sizeof(RuntimeFunction));
-    for (const RuntimeFunction *I = RFs.begin(), *E = RFs.end(); I < E; ++I) {
-      const uint64_t SectionOffset = std::distance(RFs.begin(), I)
-                                     * sizeof(RuntimeFunction);
+    for (const RuntimeFunction &RF : RFs) {
+      const uint64_t SectionOffset =
+          std::distance(RFs.begin(), &RF) * sizeof(RuntimeFunction);
 
       outs() << "Function Table:\n";
 
       outs() << "  Start Address: ";
-      printCOFFSymbolAddress(outs(), Rels, SectionOffset +
-                             /*offsetof(RuntimeFunction, StartAddress)*/ 0,
-                             I->StartAddress);
+      printCOFFSymbolAddress(outs(), Rels,
+                             SectionOffset +
+                                 /*offsetof(RuntimeFunction, StartAddress)*/ 0,
+                             RF.StartAddress);
       outs() << "\n";
 
       outs() << "  End Address: ";
-      printCOFFSymbolAddress(outs(), Rels, SectionOffset +
-                             /*offsetof(RuntimeFunction, EndAddress)*/ 4,
-                             I->EndAddress);
+      printCOFFSymbolAddress(outs(), Rels,
+                             SectionOffset +
+                                 /*offsetof(RuntimeFunction, EndAddress)*/ 4,
+                             RF.EndAddress);
       outs() << "\n";
 
       outs() << "  Unwind Info Address: ";
-      printCOFFSymbolAddress(outs(), Rels, SectionOffset +
-                             /*offsetof(RuntimeFunction, UnwindInfoOffset)*/ 8,
-                             I->UnwindInfoOffset);
+      printCOFFSymbolAddress(
+          outs(), Rels, SectionOffset +
+                            /*offsetof(RuntimeFunction, UnwindInfoOffset)*/ 8,
+          RF.UnwindInfoOffset);
       outs() << "\n";
 
       ArrayRef<uint8_t> XContents;
@@ -445,8 +448,9 @@ void llvm::printCOFFUnwindInfo(const COFFObjectFile *Obj) {
                                    XContents, UnwindInfoOffset))) continue;
       if (XContents.empty()) continue;
 
-      UnwindInfoOffset += I->UnwindInfoOffset;
-      if (UnwindInfoOffset > XContents.size()) continue;
+      UnwindInfoOffset += RF.UnwindInfoOffset;
+      if (UnwindInfoOffset > XContents.size())
+        continue;
 
       const Win64EH::UnwindInfo *UI =
                             reinterpret_cast<const Win64EH::UnwindInfo *>