[PATCH] D83875: [XCOFF][AIX] Enable -ffunction-sections

Xiangling Liao via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 27 09:05:43 PDT 2020


Xiangling_L added inline comments.


================
Comment at: llvm/lib/Target/PowerPC/PPCISelLowering.cpp:5348
           const XCOFF::StorageClass &SC) {
         MCSymbolXCOFF *S = getFunctionEntryPointSymbol(FuncName);
 
----------------
I am wondering why we don't just use `Subtarget.getTargetMachine().getObjFileLowering().getFunctionEntryPointSymbol` instead?  And all the logic below about how we get the `MCSymbolXCOFF` should be simplified by doing this?


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

https://reviews.llvm.org/D83875





More information about the llvm-commits mailing list