[PATCH] D103166: [AMDGPU] Fix function calls with flat scratch
Sebastian Neubauer via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu May 27 09:36:54 PDT 2021
sebastian-ne updated this revision to Diff 348312.
sebastian-ne added a comment.
In D103166#2782087 <https://reviews.llvm.org/D103166#2782087>, @foad wrote:
> Does globalisel handle this?
Yes, it seems like globalisel handles adding the stack pointer in the `AMDGPUOutgoingArgHandler`. The code that adds the stack pointer in `AMDGPUInstructionSelector::selectMUBUFScratchOffset` is never hit, I’ll remove it.
I changed the handling in SDAG to match the one in GlobalISel. The stack pointer is now added when lowering call arguments and not when selecting instructions.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D103166/new/
https://reviews.llvm.org/D103166
Files:
llvm/lib/Target/AMDGPU/AMDGPUISelDAGToDAG.cpp
llvm/lib/Target/AMDGPU/AMDGPUISelLowering.cpp
llvm/lib/Target/AMDGPU/AMDGPUInstructionSelector.cpp
llvm/lib/Target/AMDGPU/SIISelLowering.cpp
llvm/test/CodeGen/AMDGPU/callee-special-input-vgprs-packed.ll
llvm/test/CodeGen/AMDGPU/callee-special-input-vgprs.ll
llvm/test/CodeGen/AMDGPU/gfx-callable-argument-types.ll
llvm/test/CodeGen/AMDGPU/non-entry-alloca.ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D103166.348312.patch
Type: text/x-patch
Size: 33937 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210527/09c8dc02/attachment.bin>
More information about the llvm-commits
mailing list