[llvm-readobj] Fix "Section" output when emitting relocations in JSON

[paulkirth]

Summary

Prior to this patch, the JSON output would emit an invalid key from the
shared LLVM implementation. This caused llvm-readobj to output invalid
JSON. This patch introduces a small helper function to print the
relocation information differently between the LLVM and JSON formats.

Before this patch:

"Relocations": [Section (2) .rel.text {

  {
    "Relocation": {
      "Offset": 0,
      "Type": {
        "Value": "R_X86_64_NONE",
        "RawValue": 0
      },
      "Symbol": {
        "Value": "rel_0",
        "RawValue": 1
      }
    }
  },
  ...

After this patch:

"Relocations": [
   {
     "SectionIdx": 2,
     "Relocs": [
       {
         "Relocation": {
           "Offset": 0,
           "Type": {
             "Name": "R_X86_64_NONE",
             "Value": 0
           },
           "Symbol": {
             "Name": "rel_0",
             "Value": 1
           }
         }
       },
       ...

[jhenderson]

Please provide before and after examples, and test cases for the fix. Also, please fix the title to use "llvm-readobj" in the commit message/patch description, not simply "readobj".

[paulkirth]

Rebase

• Update summary
• Add tests
• Simplify implementation to avoid code duplication

[paulkirth]

Rebase to fix patch application

[paulkirth]

Rebase

[paulkirth]

Rebase

[paulkirth]

Rebase.

[aidengrossman]

llvm/tools/llvm-readobj/ELFDumper.cpp

6786

If this section is already getting moved around, it might be worth it to transition to a DictScope here as well (similar to what I did in D143310, but with proper string formatting logic) to leave all the formatting in a centralized place and simplify this function.

[paulkirth]

Rebase.

[paulkirth]

Address comments

• Use scoped dict to simplify implementation and be more consistent

[paulkirth]

Rebase.

[jhenderson]

llvm/tools/llvm-readobj/ELFDumper.cpp

6783

I wonder if this would be better named printRelocationSectionInfo, since it's not printing any info about the relocations themselves.

7732

Any particular reason we don't just name this key "SectionIndex" or similar?

[paulkirth]

llvm/tools/llvm-readobj/ELFDumper.cpp

6783

That's a good point. I'll update this to reflect.

7732

No, I don't recall any a particular reason. I just went w/ the Idx suffix since it was still clear, and is a pretty common way to shorten or identify index variables. I don't have a preference here so I can change this when I address the other comments.

[paulkirth]

Rebase, address comments, and update tests.

[paulkirth]

Move test code into existing test file

[paulkirth]

Rebase

[paulkirth]

Fix bad rebase between two stacked diffs. I accidentally merged the Group section commit into this one, this commit should restore the correct split between the two.

[paulkirth]

Fix formatting

[jhenderson]

Looks good, with one nit.

llvm/tools/llvm-readobj/ELFDumper.cpp

6788

Nit: blank line between this and the next function.

[paulkirth]

Add missing blank line between functions.

[paulkirth]

Rebase and remove const qualifier from StringRef.

[paulkirth]

Add back funcion removed when rebasing

[paulkirth]

Add missing blank line.

llvm/test/tools/llvm-readobj/ELF/relocations.test

 # LLVM-EXPAND-64-NEXT:]
 
 ## Show that JSON relocations are always expanded.
 # RUN: llvm-readobj -r %t64 --elf-output-style=JSON --pretty-print \
 # RUN:   | FileCheck %s --check-prefix=JSON-EXPAND-64
 # RUN: llvm-readobj -r --expand-relocs %t64 --elf-output-style=JSON --pretty-print \
 # RUN:   | FileCheck %s --check-prefix=JSON-EXPAND-64
 
-# JSON-EXPAND-64:      "Relocations": [Section (2) .rel.text {
-# JSON-EXPAND-64:        "Relocation": {
-# JSON-EXPAND-64-NEXT:     "Offset": 0,
-# JSON-EXPAND-64-NEXT:     "Type": {
-# JSON-EXPAND-64-NEXT:       "Name": "R_X86_64_NONE",
-# JSON-EXPAND-64-NEXT:       "Value": 0
-# JSON-EXPAND-64-NEXT:     },
-# JSON-EXPAND-64-NEXT:     "Symbol": {
-# JSON-EXPAND-64-NEXT:       "Name": "rel_0",
-# JSON-EXPAND-64-NEXT:       "Value": 1
-# JSON-EXPAND-64-NEXT:     }
-# JSON-EXPAND-64-NEXT:   }
-# JSON-EXPAND-64-NEXT: },
-# JSON-EXPAND-64-NEXT: {
-# JSON-EXPAND-64-NEXT:   "Relocation": {
-# JSON-EXPAND-64-NEXT:     "Offset": 1,
-# JSON-EXPAND-64-NEXT:     "Type": {
-# JSON-EXPAND-64-NEXT:       "Name": "R_X86_64_PC32",
-# JSON-EXPAND-64-NEXT:       "Value": 2
-# JSON-EXPAND-64-NEXT:     },
-# JSON-EXPAND-64-NEXT:     "Symbol": {
-# JSON-EXPAND-64-NEXT:       "Name": "rel_neg",
-# JSON-EXPAND-64-NEXT:       "Value": 2
-# JSON-EXPAND-64-NEXT:     }
-# JSON-EXPAND-64-NEXT:   }
-# JSON-EXPAND-64-NEXT: },
-# JSON-EXPAND-64-NEXT: {
-# JSON-EXPAND-64-NEXT:   "Relocation": {
-# JSON-EXPAND-64-NEXT:     "Offset": 5,
-# JSON-EXPAND-64-NEXT:     "Type": {
-# JSON-EXPAND-64-NEXT:       "Name": "R_X86_64_PLT32",
-# JSON-EXPAND-64-NEXT:       "Value": 4
-# JSON-EXPAND-64-NEXT:     },
-# JSON-EXPAND-64-NEXT:     "Symbol": {
-# JSON-EXPAND-64-NEXT:       "Name": "rel_pos",
-# JSON-EXPAND-64-NEXT:       "Value": 3
-# JSON-EXPAND-64-NEXT:     }
-# JSON-EXPAND-64-NEXT:   }
-# JSON-EXPAND-64-NEXT: },
-# JSON-EXPAND-64-NEXT: {
-# JSON-EXPAND-64-NEXT:   "Relocation": {
-# JSON-EXPAND-64-NEXT:     "Offset": 9,
-# JSON-EXPAND-64-NEXT:     "Type": {
-# JSON-EXPAND-64-NEXT:       "Name": "R_X86_64_64",
-# JSON-EXPAND-64-NEXT:       "Value": 1
-# JSON-EXPAND-64-NEXT:     },
-# JSON-EXPAND-64-NEXT:     "Symbol": {
-# JSON-EXPAND-64-NEXT:       "Name": "rel_64",
-# JSON-EXPAND-64-NEXT:       "Value": 4
-# JSON-EXPAND-64-NEXT:     }
-# JSON-EXPAND-64-NEXT:   }
-# JSON-EXPAND-64-NEXT: }}
-# JSON-EXPAND-64-NEXT: Section (3) .rela.text {
-# JSON-EXPAND-64-NEXT: ,
-# JSON-EXPAND-64-NEXT: {
-# JSON-EXPAND-64-NEXT:   "Relocation": {
-# JSON-EXPAND-64-NEXT:     "Offset": 0,
-# JSON-EXPAND-64-NEXT:     "Type": {
-# JSON-EXPAND-64-NEXT:       "Name": "R_X86_64_NONE",
-# JSON-EXPAND-64-NEXT:       "Value": 0
-# JSON-EXPAND-64-NEXT:     },
-# JSON-EXPAND-64-NEXT:     "Symbol": {
-# JSON-EXPAND-64-NEXT:       "Name": "rela_0",
-# JSON-EXPAND-64-NEXT:       "Value": 5
-# JSON-EXPAND-64-NEXT:     },
-# JSON-EXPAND-64-NEXT:     "Addend": 0
-# JSON-EXPAND-64-NEXT:   }
-# JSON-EXPAND-64-NEXT: },
-# JSON-EXPAND-64-NEXT: {
-# JSON-EXPAND-64-NEXT:   "Relocation": {
-# JSON-EXPAND-64-NEXT:     "Offset": 1,
-# JSON-EXPAND-64-NEXT:     "Type": {
-# JSON-EXPAND-64-NEXT:       "Name": "R_X86_64_PC32",
-# JSON-EXPAND-64-NEXT:       "Value": 2
-# JSON-EXPAND-64-NEXT:     },
-# JSON-EXPAND-64-NEXT:     "Symbol": {
-# JSON-EXPAND-64-NEXT:       "Name": "rela_neg",
-# JSON-EXPAND-64-NEXT:       "Value": 6
-# JSON-EXPAND-64-NEXT:     },
-# JSON-EXPAND-64-NEXT:     "Addend": 18446744073709551615
-# JSON-EXPAND-64-NEXT:   }
-# JSON-EXPAND-64-NEXT: },
-# JSON-EXPAND-64-NEXT: {
-# JSON-EXPAND-64-NEXT:   "Relocation": {
-# JSON-EXPAND-64-NEXT:     "Offset": 5,
-# JSON-EXPAND-64-NEXT:     "Type": {
-# JSON-EXPAND-64-NEXT:       "Name": "R_X86_64_PLT32",
-# JSON-EXPAND-64-NEXT:       "Value": 4
-# JSON-EXPAND-64-NEXT:     },
-# JSON-EXPAND-64-NEXT:     "Symbol": {
-# JSON-EXPAND-64-NEXT:       "Name": "rela_pos",
-# JSON-EXPAND-64-NEXT:       "Value": 7
-# JSON-EXPAND-64-NEXT:     },
-# JSON-EXPAND-64-NEXT:     "Addend": 2
-# JSON-EXPAND-64-NEXT:   }
-# JSON-EXPAND-64-NEXT: },
-# JSON-EXPAND-64-NEXT: {
-# JSON-EXPAND-64-NEXT:   "Relocation": {
-# JSON-EXPAND-64-NEXT:     "Offset": 18446744073709551615,
-# JSON-EXPAND-64-NEXT:     "Type": {
-# JSON-EXPAND-64-NEXT:       "Name": "R_X86_64_64",
-# JSON-EXPAND-64-NEXT:       "Value": 1
-# JSON-EXPAND-64-NEXT:     },
-# JSON-EXPAND-64-NEXT:     "Symbol": {
-# JSON-EXPAND-64-NEXT:       "Name": "rela_minneg",
-# JSON-EXPAND-64-NEXT:       "Value": 8
-# JSON-EXPAND-64-NEXT:     },
-# JSON-EXPAND-64-NEXT:     "Addend": 9223372036854775808
-# JSON-EXPAND-64-NEXT:   }
-# JSON-EXPAND-64-NEXT: },
-# JSON-EXPAND-64-NEXT: {
-# JSON-EXPAND-64-NEXT:   "Relocation": {
-# JSON-EXPAND-64-NEXT:     "Offset": 9,
-# JSON-EXPAND-64-NEXT:     "Type": {
-# JSON-EXPAND-64-NEXT:       "Name": "R_X86_64_32S",
-# JSON-EXPAND-64-NEXT:       "Value": 11
-# JSON-EXPAND-64-NEXT:     },
-# JSON-EXPAND-64-NEXT:     "Symbol": {
-# JSON-EXPAND-64-NEXT:       "Name": "rela_maxpos",
-# JSON-EXPAND-64-NEXT:       "Value": 9
-# JSON-EXPAND-64-NEXT:     },
-# JSON-EXPAND-64-NEXT:     "Addend": 9223372036854775807
-# JSON-EXPAND-64-NEXT:   }
-# JSON-EXPAND-64-NEXT: }}
+# JSON-EXPAND-64:      "Relocations": [
+# JSON-EXPAND-64-NEXT:   {
+# JSON-EXPAND-64-NEXT:   "SectionIndex": 2
+# JSON-EXPAND-64-NEXT:   "Relocs": [
+# JSON-EXPAND-64-NEXT:     {
+# JSON-EXPAND-64-NEXT:       "Relocation": {
+# JSON-EXPAND-64-NEXT:         "Offset": 0,
+# JSON-EXPAND-64-NEXT:         "Type": {
+# JSON-EXPAND-64-NEXT:           "Name": "R_X86_64_NONE",
+# JSON-EXPAND-64-NEXT:           "Value": 0
+# JSON-EXPAND-64-NEXT:         },
+# JSON-EXPAND-64-NEXT:         "Symbol": {
+# JSON-EXPAND-64-NEXT:           "Name": "rel_0",
+# JSON-EXPAND-64-NEXT:           "Value": 1
+# JSON-EXPAND-64-NEXT:         }
+# JSON-EXPAND-64-NEXT:       }
+# JSON-EXPAND-64-NEXT:     },
+# JSON-EXPAND-64-NEXT:     {
+# JSON-EXPAND-64-NEXT:       "Relocation": {
+# JSON-EXPAND-64-NEXT:         "Offset": 1,
+# JSON-EXPAND-64-NEXT:         "Type": {
+# JSON-EXPAND-64-NEXT:           "Name": "R_X86_64_PC32",
+# JSON-EXPAND-64-NEXT:           "Value": 2
+# JSON-EXPAND-64-NEXT:         },
+# JSON-EXPAND-64-NEXT:         "Symbol": {
+# JSON-EXPAND-64-NEXT:           "Name": "rel_neg",
+# JSON-EXPAND-64-NEXT:           "Value": 2
+# JSON-EXPAND-64-NEXT:         }
+# JSON-EXPAND-64-NEXT:       }
+# JSON-EXPAND-64-NEXT:     },
+# JSON-EXPAND-64-NEXT:     {
+# JSON-EXPAND-64-NEXT:       "Relocation": {
+# JSON-EXPAND-64-NEXT:         "Offset": 5,
+# JSON-EXPAND-64-NEXT:         "Type": {
+# JSON-EXPAND-64-NEXT:           "Name": "R_X86_64_PLT32",
+# JSON-EXPAND-64-NEXT:           "Value": 4
+# JSON-EXPAND-64-NEXT:         },
+# JSON-EXPAND-64-NEXT:         "Symbol": {
+# JSON-EXPAND-64-NEXT:           "Name": "rel_pos",
+# JSON-EXPAND-64-NEXT:           "Value": 3
+# JSON-EXPAND-64-NEXT:         }
+# JSON-EXPAND-64-NEXT:       }
+# JSON-EXPAND-64-NEXT:     },
+# JSON-EXPAND-64-NEXT:     {
+# JSON-EXPAND-64-NEXT:       "Relocation": {
+# JSON-EXPAND-64-NEXT:         "Offset": 9,
+# JSON-EXPAND-64-NEXT:         "Type": {
+# JSON-EXPAND-64-NEXT:           "Name": "R_X86_64_64",
+# JSON-EXPAND-64-NEXT:           "Value": 1
+# JSON-EXPAND-64-NEXT:         },
+# JSON-EXPAND-64-NEXT:         "Symbol": {
+# JSON-EXPAND-64-NEXT:           "Name": "rel_64",
+# JSON-EXPAND-64-NEXT:           "Value": 4
+# JSON-EXPAND-64-NEXT:         }
+# JSON-EXPAND-64-NEXT:       }
+# JSON-EXPAND-64-NEXT:     }
+# JSON-EXPAND-64-NEXT:    ]
+# JSON-EXPAND-64-NEXT:   },
+# JSON-EXPAND-64-NEXT:   {
+# JSON-EXPAND-64-NEXT:   "SectionIndex": 3,
+# JSON-EXPAND-64-NEXT:   "Relocs": [
+# JSON-EXPAND-64-NEXT:     {
+# JSON-EXPAND-64-NEXT:       "Relocation": {
+# JSON-EXPAND-64-NEXT:         "Offset": 0,
+# JSON-EXPAND-64-NEXT:         "Type": {
+# JSON-EXPAND-64-NEXT:           "Name": "R_X86_64_NONE",
+# JSON-EXPAND-64-NEXT:           "Value": 0
+# JSON-EXPAND-64-NEXT:         },
+# JSON-EXPAND-64-NEXT:         "Symbol": {
+# JSON-EXPAND-64-NEXT:           "Name": "rela_0",
+# JSON-EXPAND-64-NEXT:           "Value": 5
+# JSON-EXPAND-64-NEXT:         },
+# JSON-EXPAND-64-NEXT:         "Addend": 0
+# JSON-EXPAND-64-NEXT:       }
+# JSON-EXPAND-64-NEXT:     },
+# JSON-EXPAND-64-NEXT:     {
+# JSON-EXPAND-64-NEXT:       "Relocation": {
+# JSON-EXPAND-64-NEXT:         "Offset": 1,
+# JSON-EXPAND-64-NEXT:         "Type": {
+# JSON-EXPAND-64-NEXT:           "Name": "R_X86_64_PC32",
+# JSON-EXPAND-64-NEXT:           "Value": 2
+# JSON-EXPAND-64-NEXT:         },
+# JSON-EXPAND-64-NEXT:         "Symbol": {
+# JSON-EXPAND-64-NEXT:           "Name": "rela_neg",
+# JSON-EXPAND-64-NEXT:           "Value": 6
+# JSON-EXPAND-64-NEXT:         },
+# JSON-EXPAND-64-NEXT:         "Addend": 18446744073709551615
+# JSON-EXPAND-64-NEXT:       }
+# JSON-EXPAND-64-NEXT:     },
+# JSON-EXPAND-64-NEXT:     {
+# JSON-EXPAND-64-NEXT:       "Relocation": {
+# JSON-EXPAND-64-NEXT:         "Offset": 5,
+# JSON-EXPAND-64-NEXT:         "Type": {
+# JSON-EXPAND-64-NEXT:           "Name": "R_X86_64_PLT32",
+# JSON-EXPAND-64-NEXT:           "Value": 4
+# JSON-EXPAND-64-NEXT:         },
+# JSON-EXPAND-64-NEXT:         "Symbol": {
+# JSON-EXPAND-64-NEXT:           "Name": "rela_pos",
+# JSON-EXPAND-64-NEXT:           "Value": 7
+# JSON-EXPAND-64-NEXT:         },
+# JSON-EXPAND-64-NEXT:         "Addend": 2
+# JSON-EXPAND-64-NEXT:       }
+# JSON-EXPAND-64-NEXT:     },
+# JSON-EXPAND-64-NEXT:     {
+# JSON-EXPAND-64-NEXT:       "Relocation": {
+# JSON-EXPAND-64-NEXT:         "Offset": 18446744073709551615,
+# JSON-EXPAND-64-NEXT:         "Type": {
+# JSON-EXPAND-64-NEXT:           "Name": "R_X86_64_64",
+# JSON-EXPAND-64-NEXT:           "Value": 1
+# JSON-EXPAND-64-NEXT:         },
+# JSON-EXPAND-64-NEXT:         "Symbol": {
+# JSON-EXPAND-64-NEXT:           "Name": "rela_minneg",
+# JSON-EXPAND-64-NEXT:           "Value": 8
+# JSON-EXPAND-64-NEXT:         },
+# JSON-EXPAND-64-NEXT:         "Addend": 9223372036854775808
+# JSON-EXPAND-64-NEXT:       }
+# JSON-EXPAND-64-NEXT:     },
+# JSON-EXPAND-64-NEXT:     {
+# JSON-EXPAND-64-NEXT:       "Relocation": {
+# JSON-EXPAND-64-NEXT:         "Offset": 9,
+# JSON-EXPAND-64-NEXT:         "Type": {
+# JSON-EXPAND-64-NEXT:           "Name": "R_X86_64_32S",
+# JSON-EXPAND-64-NEXT:           "Value": 11
+# JSON-EXPAND-64-NEXT:         },
+# JSON-EXPAND-64-NEXT:         "Symbol": {
+# JSON-EXPAND-64-NEXT:           "Name": "rela_maxpos",
+# JSON-EXPAND-64-NEXT:           "Value": 9
+# JSON-EXPAND-64-NEXT:         },
+# JSON-EXPAND-64-NEXT:         "Addend": 9223372036854775807
+# JSON-EXPAND-64-NEXT:       }
+# JSON-EXPAND-64-NEXT:     }
+# JSON-EXPAND-64-NEXT:    ]
+# JSON-EXPAND-64-NEXT:  }
+# JSON-EXPAND-64-NEXT: ]
 
 --- !ELF
 FileHeader:
   Class:   ELFCLASS64
   Data:    ELFDATA2LSB
   Type:    ET_REL
   Machine: EM_X86_64
 Sections:

llvm/tools/llvm-readobj/ELFDumper.cpp

 protected:
   void printSymbolOtherField(const Elf_Sym &Symbol) const;
   virtual void printExpandedRelRelaReloc(const Relocation<ELFT> &R,
                                          StringRef SymbolName,
                                          StringRef RelocName);
   virtual void printDefaultRelRelaReloc(const Relocation<ELFT> &R,
                                         StringRef SymbolName,
                                         StringRef RelocName);
+  virtual void printRelocationSectionInfo(const Elf_Shdr &Sec, StringRef Name,
+                                          const unsigned SecNdx);
 
   ScopedPrinter &W;
 };
 
 // JSONELFDumper shares most of the same implementation as LLVMELFDumper except
 // it uses a JSONScopedPrinter.
 template <typename ELFT> class JSONELFDumper : public LLVMELFDumper<ELFT> {
 public:
   LLVM_ELF_IMPORT_TYPES_ELFT(ELFT)
 
   JSONELFDumper(const object::ELFObjectFile<ELFT> &ObjF, ScopedPrinter &Writer)
       : LLVMELFDumper<ELFT>(ObjF, Writer) {}
 
   void printFileSummary(StringRef FileStr, ObjectFile &Obj,
                         ArrayRef<std::string> InputFilenames,
                         const Archive *A) override;
   virtual void printZeroSymbolOtherField(const Elf_Sym &Symbol) const override;
 
   void printDefaultRelRelaReloc(const Relocation<ELFT> &R,
                                 StringRef SymbolName,
                                 StringRef RelocName) override;
 
+  void printRelocationSectionInfo(const Elf_Shdr &Sec, StringRef Name,
+                                  const unsigned SecNdx) override;
+
 private:
   std::unique_ptr<DictScope> FileScope;
 };
 
 } // end anonymous namespace
 
 namespace llvm {
 
template <class ELFT> void LLVMELFDumper<ELFT>::printRelocations() {
   ListScope D(W, "Relocations");
 
   for (const Elf_Shdr &Sec : cantFail(this->Obj.sections())) {
     if (!isRelocationSec<ELFT>(Sec))
       continue;
 
     StringRef Name = this->getPrintableSectionName(Sec);
     unsigned SecNdx = &Sec - &cantFail(this->Obj.sections()).front();
-    W.startLine() << "Section (" << SecNdx << ") " << Name << " {\n";
-    W.indent();
-    this->printRelocationsHelper(Sec);
-    W.unindent();
-    W.startLine() << "}\n";
+    printRelocationSectionInfo(Sec, Name, SecNdx);
   }
 }
 
 template <class ELFT>
 void LLVMELFDumper<ELFT>::printRelrReloc(const Elf_Relr &R) {
   W.startLine() << W.hex(R) << "\n";
 }
 
void LLVMELFDumper<ELFT>::printDefaultRelRelaReloc(const Relocation<ELFT> &R,
   OS << W.hex(R.Offset) << " " << RelocName << " "
      << (!SymbolName.empty() ? SymbolName : "-");
   if (R.Addend)
     OS << " " << W.hex((uintX_t)*R.Addend);
   OS << "\n";
 }
 
 template <class ELFT>
+void LLVMELFDumper<ELFT>::printRelocationSectionInfo(const Elf_Shdr &Sec,
+                                                     StringRef Name,
+                                                     const unsigned SecNdx) {
+  DictScope D(W, (Twine("Section (") + Twine(SecNdx) + ") " + Name).str());
+  this->printRelocationsHelper(Sec);
+}
+
+template <class ELFT>
 void LLVMELFDumper<ELFT>::printRelRelaReloc(const Relocation<ELFT> &R,
                                             const RelSymbol<ELFT> &RelSym) {
   StringRef SymbolName = RelSym.Name;
   SmallString<32> RelocName;
   this->Obj.getRelocationTypeName(R.Type, RelocName);
 
   if (opts::ExpandRelocs) {
     printExpandedRelRelaReloc(R, SymbolName, RelocName);

 }
 
 template <class ELFT>
 void JSONELFDumper<ELFT>::printDefaultRelRelaReloc(const Relocation<ELFT> &R,
                                                    StringRef SymbolName,
                                                    StringRef RelocName) {
   this->printExpandedRelRelaReloc(R, SymbolName, RelocName);
 }
+
+template <class ELFT>
+void JSONELFDumper<ELFT>::printRelocationSectionInfo(const Elf_Shdr &Sec,
+                                                     StringRef Name,
+                                                     const unsigned SecNdx) {
+  DictScope Group(this->W);
+  this->W.printNumber("SectionIndex", SecNdx);
+  ListScope D(this->W, "Relocs");
+  this->printRelocationsHelper(Sec);
+}