diff options
| author | Matt Arsenault <Matthew.Arsenault@amd.com> | 2014-09-26 17:54:46 +0000 |
|---|---|---|
| committer | Matt Arsenault <Matthew.Arsenault@amd.com> | 2014-09-26 17:54:46 +0000 |
| commit | 0bea8d830e4f2eb1db9d04b3579581e09a794dc0 (patch) | |
| tree | 02e4375c3ed2e7d305d900fa1953d43b3bac4c53 /llvm/lib | |
| parent | aff65fbca5616ad4652c60ab292db978bdad60f6 (diff) | |
| download | bcm5719-llvm-0bea8d830e4f2eb1db9d04b3579581e09a794dc0.tar.gz bcm5719-llvm-0bea8d830e4f2eb1db9d04b3579581e09a794dc0.zip | |
R600/SI: Don't assert on exotic operand types
This needs a test, but I'm not sure if it is currently possible and
I originally hit it due to a bug. Right now the only global address
operands have no reason to be VALU instructions, although it
theoretically could be a problem.
llvm-svn: 218528
Diffstat (limited to 'llvm/lib')
| -rw-r--r-- | llvm/lib/Target/R600/SIInstrInfo.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/Target/R600/SIInstrInfo.cpp b/llvm/lib/Target/R600/SIInstrInfo.cpp index 3ad448ee421..bc3f9ddf1df 100644 --- a/llvm/lib/Target/R600/SIInstrInfo.cpp +++ b/llvm/lib/Target/R600/SIInstrInfo.cpp @@ -705,7 +705,7 @@ MachineInstr *SIInstrInfo::commuteInstruction(MachineInstr *MI, if (Src1Idx != -1 && !MI->getOperand(Src1Idx).isReg()) { // XXX: Commute instructions with FPImm operands - if (NewMI || MI->getOperand(Src1Idx).isFPImm() || + if (NewMI || !MI->getOperand(Src1Idx).isImm() || (!isVOP2(MI->getOpcode()) && !isVOP3(MI->getOpcode()))) { return nullptr; } |
