]> granicus.if.org Git - llvm/commit
[AMDGPU] Fix-up cases where writelane has 2 SGPR operands
authorDavid Stuttard <david.stuttard@amd.com>
Wed, 16 Oct 2019 14:37:39 +0000 (14:37 +0000)
committerDavid Stuttard <david.stuttard@amd.com>
Wed, 16 Oct 2019 14:37:39 +0000 (14:37 +0000)
commit9cb56c603f5e64105aa75e32f1d8400ebfcb0d46
tree44c71df620eb8ad0a8949fabfe75fd3ea87cb326
parent4f04974098b2104ceb1ba74c58e2ca725bb7553c
[AMDGPU] Fix-up cases where writelane has 2 SGPR operands

Summary:
Even though writelane doesn't have the same constraints as other valu
instructions it still can't violate the >1 SGPR operand constraint

Due to later register propagation (e.g. fixing up vgpr operands via
readfirstlane) changing writelane to only have a single SGPR is tricky.

This implementation puts a new check after SIFixSGPRCopies that prevents
multiple SGPRs being used in any writelane instructions.

The algorithm used is to check for trivial copy prop of suitable constants into
one of the SGPR operands and perform that if possible. If this isn't possible
put an explicit copy of Src1 SGPR into M0 and use that instead (this is
allowable for writelane as the constraint is for SGPR read-port and not
constant-bus access).

Reviewers: rampitec, tpr, arsenm, nhaehnle

Reviewed By: rampitec, arsenm, nhaehnle

Subscribers: arsenm, kzhuravl, jvesely, wdng, nhaehnle, mgorny, yaxunl, tpr, t-tye, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D51932

Change-Id: Ic7553fa57440f208d4dbc4794fc24345d7e0e9ea

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@375004 91177308-0d34-0410-b5e6-96231b3b80d8
lib/Target/AMDGPU/SIFixSGPRCopies.cpp
lib/Target/AMDGPU/SIInstrInfo.cpp
test/CodeGen/AMDGPU/inserted-wait-states.mir
test/CodeGen/AMDGPU/llvm.amdgcn.writelane.ll