[PATCH] D17523: llvm-readobj: enable GNU output style sections and relocations printing for ELF files

[khemant@codeaurora.org via llvm-commits]

khemant added inline comments.

================
Comment at: tools/llvm-readobj/ELFDumper.cpp:237
@@ -222,2 +236,3 @@
 public:
-  virtual void printFileHeaders(const ELFFile<ELFT> *Obj) = 0;
+  TYPEDEF_ELF_TYPES(ELFT)
+  virtual void printFileHeaders(const ELFO *Obj) = 0;
----------------
rafael wrote:
> Do you need the typedef here?
Correct. This is not needed in base class as I only use object here.

================
Comment at: tools/llvm-readobj/ELFDumper.cpp:2297
@@ +2296,3 @@
+    OS << "\nRelocation section '" << Name << "' at offset 0x"
+       << to_hexString(Offset, false) << " contains " << to_string(Entries)
+       << " entries:\n";
----------------
rafael wrote:
> why do you need the to_string here?
Correct. This is no longer needed.


Repository:
  rL LLVM

http://reviews.llvm.org/D17523