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

[James Henderson via Phabricator via llvm-commits]

jhenderson added inline comments.


================
Comment at: llvm/tools/llvm-readobj/ELFDumper.cpp:6724
+template <class ELFT>
+void LLVMELFDumper<ELFT>::printRelocationInfo(const Elf_Shdr &Sec,
+                                              const StringRef Name,
----------------
I wonder if this would be better named `printRelocationSectionInfo`, since it's not printing any info about the relocations themselves.


================
Comment at: llvm/tools/llvm-readobj/ELFDumper.cpp:7635
+  DictScope Group(this->W);
+  this->W.printNumber("SectionIdx", SecNdx);
+  ListScope D(this->W, "Relocs");
----------------
Any particular reason we don't just name this key "SectionIndex" or similar?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D137094/new/

https://reviews.llvm.org/D137094