[PATCH] D62179: [llvm-readelf] - Allow dumping of the .dynamic section even if there is no PT_DYNAMIC header.

[George Rimar via Phabricator via llvm-commits]

grimar marked an inline comment as done.
grimar added inline comments.


================
Comment at: tools/llvm-readobj/ELFDumper.cpp:1486
       HashTable =
-          reinterpret_cast<const Elf_Hash *>(toMappedAddr(Dyn.getPtr()));
+          reinterpret_cast<const Elf_Hash *>(toMappedAddr("DT_HASH", Dyn.getPtr()));
       break;
----------------
rupprecht wrote:
> Instead of constants like `"DT_HASH"` for all these changes, can you pass in `Dyn.d_tag` instead and use `getTypeString()` within `toMappedAddr` to print the string? (Passing `Obj->getHeader()->e_machine` by lambda capture). That will keep all these calls identical.
Oh, cool, I did not know we have something like `getTypeString()` here (or perhaps forgot about that) , thanks!


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

https://reviews.llvm.org/D62179