[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
Thu Mar 18 14:09:42 PDT 2021


DiggerLin added inline comments.


================
Comment at: llvm/lib/Object/XCOFFObjectFile.cpp:815
 
-  // A function definition should be a label definition.
-  if ((CsectAuxEnt->SymbolAlignmentAndType & SymTypeMask) != XCOFF::XTY_LD)
-    return false;
+  auto getSymbolAuxType = [this](uintptr_t AuxAddr) {
+    return viewAs<XCOFF::SymbolAuxType>(
----------------
create  a static function getSymbolAuxType in a file scope maybe  better? 
All the Aux symbol of 64bit all has the AuxType .  we can use the function for other type too in other place later ?


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

https://reviews.llvm.org/D85774



More information about the llvm-commits mailing list