[PATCH] D115551: [AMDGPU] Do not reserve any VGPR for SGPR spills

[Austin Kerbow via Phabricator via llvm-commits]

kerbowa added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/SIFrameLowering.cpp:1321-1323
-  const bool WillHaveFP =
-      FrameInfo.hasCalls() && (HaveAnyCSRVGPR || MFI->VGPRReservedForSGPRSpill);
 
----------------
arsenm wrote:
> I don't see why you're changing this, this is still an issue?
With the change, we are being more conservative.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115551