[PATCH] D99269: [AMDGPU] Unify spill code
Matt Arsenault via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Mar 30 15:28:54 PDT 2021
arsenm requested changes to this revision.
arsenm added inline comments.
This revision now requires changes to proceed.
================
Comment at: llvm/lib/Target/AMDGPU/SIRegisterInfo.cpp:1456-1459
+ // Use the stack pointer instead of the frame pointer for restores in the
+ // function epilog.
+ if (Flags & 1)
+ FrameReg = MFI->getStackPtrOffsetReg();
----------------
I don't like having this logic separated and overriding the frame register logic above. Can you infer this should be SP relative from the frame index itself instead of adding a new operand?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D99269/new/
https://reviews.llvm.org/D99269
More information about the llvm-commits
mailing list