[PATCH] D18224: llvm-readobj: GNU style symbols printing

khemant@codeaurora.org via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 17 15:39:57 PDT 2016


khemant added a comment.

The patch to upstream the processor specific flags for small area is not yet in the Hexagon backend. That is why I made a binary. I do not know of other backends that can get me this flag.


================
Comment at: tools/llvm-readobj/ELFDumper.cpp:47
@@ -46,1 +46,3 @@
 
+#define ELF_VISIBILITY(other) ((other)&0x3)
+
----------------
rafael wrote:
> There is a getVisibility in Elf_Sym.
Got it. Will use it

================
Comment at: tools/llvm-readobj/ELFDumper.cpp:184
@@ -184,2 +183,3 @@
   const Elf_Shdr *DotSymtabSec = nullptr;
+  const Elf_Shdr *DotDynSymtabSec = nullptr;
   ArrayRef<Elf_Word> ShndxTable;
----------------
rafael wrote:
> Please avoid adding this.
> 
> You can use dynamic_symbols.
> 
I need a way to get the name of the symbol table (both static and dynamic) when the section is present. Any ideas how do I do it?




Repository:
  rL LLVM

http://reviews.llvm.org/D18224





More information about the llvm-commits mailing list