[PATCH] D84231: [llvm-readobj] - Don't get the name of the symbol table in ELFDumper<ELFT>::printSymbolsHelper.

Wed Jul 22 01:09:21 PDT 2020

jhenderson added inline comments.

================
Comment at: llvm/test/Object/invalid.test:292
+# INVALID-SECTION-SIZE2-NEXT:  Other: 0
+# INVALID-SECTION-SIZE2-NEXT: warning: '[[FILE]]': section [index 1] has a sh_offset (0xffffffff) + sh_size (0x1b) that cannot be represented
+# INVALID-SECTION-SIZE2-NEXT:  Section: <?> (0x1)
----------------
This seems to be missing context that was useful before? In other words, what went wrong here?

================
Comment at: llvm/tools/llvm-readobj/ELFDumper.cpp:3982
+      this->reportUniqueWarning(createError(
+          "unable to get the name of the symbol table section with index " +
+          Twine(SecNdx) + ": " + toString(NameOrErr.takeError())));
----------------
MaskRay wrote:
> The original messages carry more information as a symbol table can represent either  SHT_DYNSYM or SHT_SYMTAB... I am fine combing them, though
Could you use the new `describe` function from D84240 to resolve that?

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

https://reviews.llvm.org/D84231