[X86] commuteInstructionImpl - assert that only MOVSDrr is being commuted to SHUFPDrri

RKSimon · RKSimon · commit 054646f335e8 · 2025-06-10T08:49:18.000+01:00
Noticed while preparing for #142972
diff --git a/llvm/lib/Target/X86/X86InstrInfo.cpp b/llvm/lib/Target/X86/X86InstrInfo.cpp
@@ -2487,6 +2487,7 @@ MachineInstr *X86InstrInfo::commuteInstructionImpl(MachineInstr &MI, bool NewMI,
       break;
     }
 
+    assert(Opc == X86::MOVSDrr && "Only MOVSD can commute to SHUFPD");
     WorkingMI = CloneIfNew(MI);
     WorkingMI->setDesc(get(X86::SHUFPDrri));
     WorkingMI->addOperand(MachineOperand::CreateImm(0x02));

Original file line number	Diff line number	Diff line change
`@@ -2487,6 +2487,7 @@ MachineInstr *X86InstrInfo::commuteInstructionImpl(MachineInstr &MI, bool NewMI,`
`2487`	`2487`	`break;`
`2488`	`2488`	`}`
`2489`	`2489`
	`2490`	`+ assert(Opc == X86::MOVSDrr && "Only MOVSD can commute to SHUFPD");`
`2490`	`2491`	`WorkingMI = CloneIfNew(MI);`
`2491`	`2492`	`WorkingMI->setDesc(get(X86::SHUFPDrri));`
`2492`	`2493`	`WorkingMI->addOperand(MachineOperand::CreateImm(0x02));`