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

Jason Liu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 27 09:02:42 PDT 2020


jasonliu marked an inline comment as done.
jasonliu added inline comments.


================
Comment at: llvm/lib/CodeGen/TargetLoweringObjectFileImpl.cpp:2088
     const Function &F, const TargetMachine &TM) const {
   assert (!TM.getFunctionSections() && "Unique sections not supported on XCOFF"
           " yet.");
----------------
Xiangling_L wrote:
> Should we also handle this in this patch?
I think this is fairly orthogonal and we could handle it with a follow up patch.
Noted that without handling this, we might run into asserts when using function sections with jump table.


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

https://reviews.llvm.org/D83875





More information about the llvm-commits mailing list