[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