diff options
author | Jay Foad <jay.foad@amd.com> | 2020-02-11 19:12:48 +0000 |
---|---|---|
committer | Jay Foad <jay.foad@amd.com> | 2020-02-11 20:11:41 +0000 |
commit | 9df0c264d4e34e895208a270045b0e9ed22b658f (patch) | |
tree | d7c8ad6869ad728648d09244b1ed383f48f03220 /clang/lib/Frontend/CompilerInvocation.cpp | |
parent | 11fb68abab9594fd9e9b07a5facf5f5dbf4a4050 (diff) | |
download | llvm-9df0c264d4e34e895208a270045b0e9ed22b658f.zip llvm-9df0c264d4e34e895208a270045b0e9ed22b658f.tar.gz llvm-9df0c264d4e34e895208a270045b0e9ed22b658f.tar.bz2 |
[AMDGPU] Fix implicit operands for ENTER_WWM pseudo
Summary:
SIInstrInfo::expandPostRAPseudo converts ENTER_WWM in-place into an
S_OR_SAVEEXEC instruction that needs certain implicit operands. Without
this patch I get errors like this that make it harder to use -stop-after
to bisect the pass pipeline:
$ llc -march=amdgcn test/CodeGen/AMDGPU/wqm.ll -stop-after=postrapseudos -o - | sed -E 's/ (from|into) custom "TargetCustom[0-9]+"//' | llc -march=amdgcn -x=mir
error: <stdin>:1295:70: missing implicit register operand 'implicit-def $scc'
renamable $sgpr2_sgpr3 = S_OR_SAVEEXEC_B64 -1, implicit-def $exec
^
Note that this error is currently only generated by MIParser but it
comes with a FIXME comment:
// FIXME: Move the implicit operand verification to the machine verifier.
Reviewers: critson, arsenm, rampitec, nhaehnle
Subscribers: kzhuravl, jvesely, wdng, yaxunl, dstuttard, tpr, t-tye, hiraditya, kerbowa, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D74428
Diffstat (limited to 'clang/lib/Frontend/CompilerInvocation.cpp')
0 files changed, 0 insertions, 0 deletions