diff options
Diffstat (limited to 'llvm')
| -rw-r--r-- | llvm/lib/Target/AMDGPU/SIInstrInfo.cpp | 18 | ||||
| -rw-r--r-- | llvm/test/CodeGen/AMDGPU/verify-sop.mir | 21 |
2 files changed, 39 insertions, 0 deletions
diff --git a/llvm/lib/Target/AMDGPU/SIInstrInfo.cpp b/llvm/lib/Target/AMDGPU/SIInstrInfo.cpp index 8f4a073839d..1e77d63ba9d 100644 --- a/llvm/lib/Target/AMDGPU/SIInstrInfo.cpp +++ b/llvm/lib/Target/AMDGPU/SIInstrInfo.cpp @@ -3191,6 +3191,24 @@ bool SIInstrInfo::verifyInstruction(const MachineInstr &MI, } } + if (isSOP2(MI) || isSOPC(MI)) { + const MachineOperand &Src0 = MI.getOperand(Src0Idx); + const MachineOperand &Src1 = MI.getOperand(Src1Idx); + unsigned Immediates = 0; + + if (!Src0.isReg() && + !isInlineConstant(Src0, Desc.OpInfo[Src0Idx].OperandType)) + Immediates++; + if (!Src1.isReg() && + !isInlineConstant(Src1, Desc.OpInfo[Src1Idx].OperandType)) + Immediates++; + + if (Immediates > 1) { + ErrInfo = "SOP2/SOPC instruction requires too many immediate constants"; + return false; + } + } + if (isSOPK(MI)) { auto Op = getNamedOperand(MI, AMDGPU::OpName::simm16); if (Desc.isBranch()) { diff --git a/llvm/test/CodeGen/AMDGPU/verify-sop.mir b/llvm/test/CodeGen/AMDGPU/verify-sop.mir new file mode 100644 index 00000000000..53d749f7119 --- /dev/null +++ b/llvm/test/CodeGen/AMDGPU/verify-sop.mir @@ -0,0 +1,21 @@ +# RUN: not llc -march=amdgcn -run-pass machineverifier %s -o - 2>&1 | FileCheck %s + +# CHECK: *** Bad machine code: SOP2/SOPC instruction requires too many immediate constants +# CHECK: - instruction: %0:sreg_32_xm0 = S_ADD_I32 + +# CHECK: *** Bad machine code: SOP2/SOPC instruction requires too many immediate constants +# CHECK: - instruction: S_CMP_EQ_U32 + +# CHECK-NOT: Bad machine code + +--- +name: sop2_sopc +tracksRegLiveness: true +body: | + bb.0: + %0:sreg_32_xm0 = S_ADD_I32 2011, -113, implicit-def $scc + S_CMP_EQ_U32 2011, -113, implicit-def $scc + + %1:sreg_32_xm0 = S_SUB_I32 2011, 10, implicit-def $scc + S_CMP_LG_U32 -5, 2011, implicit-def $scc +... |
