[PATCH] D75756: [llvm-objdump] Teach `llvm-objdump` dump dynamic symbols.

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Mar 7 08:35:26 PST 2020


MaskRay added inline comments.


================
Comment at: llvm/test/tools/llvm-objdump/elf-dynamic-symbol.test:1
+## Test that llvm-objdump can dump dynamic symbols.
+
----------------
My idea is that after we have the binary format specific directory `test/tools/llvm-objdump/ELF/`. We can move tests under it.


================
Comment at: llvm/tools/llvm-objdump/llvm-objdump.cpp:1936
       GlobLoc = Global ? 'g' : 'l';
     char Debug = (Type == SymbolRef::ST_Debug || Type == SymbolRef::ST_File)
                  ? 'd' : ' ';
----------------
For .dynsym, `char Debug` should use 'D' instead of ' '


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75756





More information about the llvm-commits mailing list