[PATCH] D58615: [llvm-objdump] Add `Version Definitions` dumper

Xing via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 25 06:23:46 PST 2019


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


================
Comment at: tools/llvm-objdump/ELFDump.cpp:313
+    const Elf_Verdef *Verdef = reinterpret_cast<const Elf_Verdef *>(Buf);
+    outs() << VerdefIndex + 1 << " "
+           << format("0x%02" PRIx16 " ", (uint16_t)Verdef->vd_flags)
----------------
grimar wrote:
> And do:
> 
> ```
> outs() << VerdefIndex++ << " "
> ```
Oh, that's cool! Thanks a lot.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D58615





More information about the llvm-commits mailing list