diff options
author | Craig Topper <craig.topper@gmail.com> | 2016-10-17 06:41:18 +0000 |
---|---|---|
committer | Craig Topper <craig.topper@gmail.com> | 2016-10-17 06:41:18 +0000 |
commit | 1f5178ff9f09f4b82f2eb3015fc3e15ac5610df0 (patch) | |
tree | 05552a8634ac80e5e050d7e0e1c875df479fa65f /llvm/lib/Target | |
parent | 9f9eee015216a19ba1c45462254c905cd9e62db9 (diff) | |
download | bcm5719-llvm-1f5178ff9f09f4b82f2eb3015fc3e15ac5610df0.tar.gz bcm5719-llvm-1f5178ff9f09f4b82f2eb3015fc3e15ac5610df0.zip |
[X86] Fix shuffle decoding assertions to print the right number of required operands. Update the checks themselves to be >= to the same number instead of > one less than the required number.
llvm-svn: 284365
Diffstat (limited to 'llvm/lib/Target')
-rw-r--r-- | llvm/lib/Target/X86/X86MCInstLower.cpp | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/llvm/lib/Target/X86/X86MCInstLower.cpp b/llvm/lib/Target/X86/X86MCInstLower.cpp index 8f4616851cd..25a9cd0ec2e 100644 --- a/llvm/lib/Target/X86/X86MCInstLower.cpp +++ b/llvm/lib/Target/X86/X86MCInstLower.cpp @@ -1534,8 +1534,8 @@ void X86AsmPrinter::EmitInstruction(const MachineInstr *MI) { case X86::VPERMILPDZrm: { if (!OutStreamer->isVerboseAsm()) break; - assert(MI->getNumOperands() > 5 && - "We should always have at least 5 operands!"); + assert(MI->getNumOperands() >= 6 && + "We should always have at least 6 operands!"); const MachineOperand &DstOp = MI->getOperand(0); const MachineOperand &SrcOp = MI->getOperand(1); const MachineOperand &MaskOp = MI->getOperand(5); @@ -1556,8 +1556,8 @@ void X86AsmPrinter::EmitInstruction(const MachineInstr *MI) { case X86::VPERMILPSZrm: { if (!OutStreamer->isVerboseAsm()) break; - assert(MI->getNumOperands() > 5 && - "We should always have at least 5 operands!"); + assert(MI->getNumOperands() >= 6 && + "We should always have at least 6 operands!"); const MachineOperand &DstOp = MI->getOperand(0); const MachineOperand &SrcOp = MI->getOperand(1); const MachineOperand &MaskOp = MI->getOperand(5); @@ -1577,8 +1577,8 @@ void X86AsmPrinter::EmitInstruction(const MachineInstr *MI) { case X86::VPERMIL2PSrmY: { if (!OutStreamer->isVerboseAsm()) break; - assert(MI->getNumOperands() > 7 && - "We should always have at least 7 operands!"); + assert(MI->getNumOperands() >= 8 && + "We should always have at least 8 operands!"); const MachineOperand &DstOp = MI->getOperand(0); const MachineOperand &SrcOp1 = MI->getOperand(1); const MachineOperand &SrcOp2 = MI->getOperand(2); @@ -1607,8 +1607,8 @@ void X86AsmPrinter::EmitInstruction(const MachineInstr *MI) { case X86::VPPERMrrm: { if (!OutStreamer->isVerboseAsm()) break; - assert(MI->getNumOperands() > 6 && - "We should always have at least 6 operands!"); + assert(MI->getNumOperands() >= 7 && + "We should always have at least 7 operands!"); const MachineOperand &DstOp = MI->getOperand(0); const MachineOperand &SrcOp1 = MI->getOperand(1); const MachineOperand &SrcOp2 = MI->getOperand(2); |