[PATCH] D151711: PowerPC/SPE: Grab the emergency slot for the vreg(that was created by the eliminateFramePointer)

Kishan Parmar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 18 05:14:51 PDT 2023


long5hot added a comment.

ping!



================
Comment at: llvm/lib/Target/PowerPC/PPCFrameLowering.cpp:2297
   if (MFI.hasVarSizedObjects() || spillsCR(MF) || hasNonRISpills(MF) ||
-      (hasSpills(MF) && !isInt<16>(StackSize))) {
+      (hasSpills(MF) && [&StackSize, &Subtarget]() -> bool {
+        if (Subtarget.hasSPE())
----------------
jhibbits wrote:
> Not a fan of including a lambda in a conditional expression.  It makes the expression as a whole harder to read.
got it!


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

https://reviews.llvm.org/D151711



More information about the llvm-commits mailing list