[PATCH] D42898: Do not spill CSR to stack on entry to noreturn functions

Francis Visoiu Mistrih via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Feb 4 22:12:24 PST 2018


thegameg added subscribers: MatzeB, thegameg.
thegameg added a comment.

Thanks for working on this!



================
Comment at: lib/CodeGen/TargetFrameLoweringImpl.cpp:90
+  if (MF.getFunction().hasFnAttribute(Attribute::NoReturn))
+    return;
+
----------------
You probably have to check for `nounwind` as well (check out @MatzeB's comment here: https://reviews.llvm.org/D42509#988553).


Repository:
  rL LLVM

https://reviews.llvm.org/D42898





More information about the llvm-commits mailing list