diff options
| author | Matt Arsenault <Matthew.Arsenault@amd.com> | 2015-09-24 07:51:25 +0000 |
|---|---|---|
| committer | Matt Arsenault <Matthew.Arsenault@amd.com> | 2015-09-24 07:51:25 +0000 |
| commit | 708586faa23d07aba2423f2c0133f6eab4f3e6ec (patch) | |
| tree | abe461f76e6864a22ef0cae5f59be5965dfa2851 | |
| parent | fa242960fc28f433f2fe4eecbe87dd4ebb569524 (diff) | |
| download | bcm5719-llvm-708586faa23d07aba2423f2c0133f6eab4f3e6ec.tar.gz bcm5719-llvm-708586faa23d07aba2423f2c0133f6eab4f3e6ec.zip | |
AMDGPU: Remove another unnecessary check from commuteInstruction
llvm-svn: 248475
| -rw-r--r-- | llvm/lib/Target/AMDGPU/SIInstrInfo.cpp | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/llvm/lib/Target/AMDGPU/SIInstrInfo.cpp b/llvm/lib/Target/AMDGPU/SIInstrInfo.cpp index e2b6d3c1403..f021e0fd4fe 100644 --- a/llvm/lib/Target/AMDGPU/SIInstrInfo.cpp +++ b/llvm/lib/Target/AMDGPU/SIInstrInfo.cpp @@ -790,17 +790,12 @@ MachineInstr *SIInstrInfo::commuteInstruction(MachineInstr *MI, int Src0Idx = AMDGPU::getNamedOperandIdx(MI->getOpcode(), AMDGPU::OpName::src0); - assert(Src0Idx != -1 && "Should always have src0 operand"); - MachineOperand &Src0 = MI->getOperand(Src0Idx); if (!Src0.isReg()) return nullptr; int Src1Idx = AMDGPU::getNamedOperandIdx(MI->getOpcode(), AMDGPU::OpName::src1); - if (Src1Idx == -1) - return nullptr; - MachineOperand &Src1 = MI->getOperand(Src1Idx); // Make sure it's legal to commute operands for VOP2. |
