[llvm] [AMDGPU] Make SIShrinkInstructions pass return valid changed state (PR #168833)

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 12 10:52:31 PST 2025


================
@@ -580,6 +591,7 @@ bool SIShrinkInstructions::shrinkScalarLogicOp(MachineInstr &MI) const {
     if (Dest->getReg().isVirtual() && SrcReg->isReg()) {
       MRI->setRegAllocationHint(Dest->getReg(), 0, SrcReg->getReg());
       MRI->setRegAllocationHint(SrcReg->getReg(), 0, Dest->getReg());
+      MoveIterator = true;
----------------
arsenm wrote:

I'd rather turn this into a return enum of statuses. It's really 2 levels of change - actually changed the instructions, and changed the function but not the actual content (only MRI register hint fields) 

https://github.com/llvm/llvm-project/pull/168833


More information about the llvm-commits mailing list