[PATCH] D81070: [AMDGPU] Fixup use of StackPtrOffsetReg when not initialized
David Stuttard via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jun 3 02:43:09 PDT 2020
dstuttard created this revision.
Herald added subscribers: llvm-commits, kerbowa, hiraditya, t-tye, tpr, yaxunl, nhaehnle, wdng, jvesely, kzhuravl, arsenm, qcolombet.
Herald added a project: LLVM.
dstuttard updated this revision to Diff 268107.
dstuttard added a comment.
dstuttard added reviewers: scott.linder, arsenm.
dstuttard added a reviewer: critson.
Improved the test to include one that has a call provoking s32 to be initialized
In some cases, StackPtrOffsetReg will not have been initialized. (In the case
where MFI.hasCalls() evaluates to false).
In this situation we can use the register, but don't need to add/subtract the
offset (it's not used in the rest of the entry function).
Change-Id: If317b29ff1e5c75a3ed3662920e6e960bca3b782
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D81070
Files:
llvm/lib/Target/AMDGPU/SIFrameLowering.cpp
llvm/lib/Target/AMDGPU/SIRegisterInfo.cpp
llvm/test/CodeGen/AMDGPU/spill-scavenge-offset.ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D81070.268107.patch
Type: text/x-patch
Size: 6265 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200603/1d7ab49e/attachment.bin>
More information about the llvm-commits
mailing list