[PATCH] D83923: [MachineOutliner][AArch64] Fix for noreturn functions

Puyan Lotfi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 20 21:16:34 PDT 2020


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


================
Comment at: llvm/lib/Target/AArch64/AArch64InstrInfo.cpp:6199
       FrameID = MachineOutlinerNoLRSave;
     } else {
       SetCandidateCallInfo(MachineOutlinerDefault, 12);
----------------
I can put the above bail-out code here in case we want to only bail if the cases are true and we are sure there isn't enough space to save the LR. @paquette what do you think here? 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83923





More information about the llvm-commits mailing list