[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