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

Puyan Lotfi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 15 18:51:12 PDT 2020


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


================
Comment at: llvm/lib/Target/AArch64/AArch64InstrInfo.cpp:6126
+      for (auto I = C.front(), E = C.back(); I != E; I++)
+        DoCandidatesHaveCallsOrNoReturn |= I->isCall();
 
----------------
kyulee wrote:
> Shouldn't `FlagsSetInAll & MachineOutlinerMBBFlags::HasCalls` cover?
Ah nice, I will use that. Thanks!


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