[PATCH] D85774: [XCOFF][AIX] Enable tooling support for 64 bit symbol table parsing

Digger Lin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 10 12:37:17 PDT 2021


DiggerLin added inline comments.


================
Comment at: llvm/tools/llvm-readobj/XCOFFDumper.cpp:379
     // error information, print out the raw Auxiliary entry from 1st till
     // the last - 1. The last one must be a CSECT Auxiliary Entry.
     for (int i = 1; i < NumberOfAuxEntries; i++) {
----------------
this is only for the 32bits .
" By convention, the csect auxiliary entry in an XCOFF32 file must be the last auxiliary entry for any external symbol that has more than one auxiliary entry"

for 64bit, it maybe look for the x_auxtype ==AUX_CSECT 


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

https://reviews.llvm.org/D85774



More information about the llvm-commits mailing list