[Legalizer] Check full condition for UMIN and UMAX just like the code below does for SMIN and SMAX (llvm#87932)

AZero13 · web-flow · commit 70bf1396517b · 2024-05-26T15:07:31.000-04:00
diff --git a/llvm/lib/CodeGen/GlobalISel/LegalizerHelper.cpp b/llvm/lib/CodeGen/GlobalISel/LegalizerHelper.cpp
@@ -3972,7 +3972,7 @@ LegalizerHelper::lower(MachineInstr &MI, unsigned TypeIdx, LLT LowerHintTy) {
     // target can override this with custom lowering and calling the
     // implementation functions.
     LLT Ty = MRI.getType(MI.getOperand(0).getReg());
-    if (LI.isLegalOrCustom({G_UMIN, Ty}))
+    if (LI.isLegalOrCustom({G_UMIN, Ty}) && LI.isLegalOrCustom({G_UMAX, Ty}))
       return lowerAddSubSatToMinMax(MI);
     return lowerAddSubSatToAddoSubo(MI);
   }

Original file line number	Diff line number	Diff line change
`@@ -3972,7 +3972,7 @@ LegalizerHelper::lower(MachineInstr &MI, unsigned TypeIdx, LLT LowerHintTy) {`
`3972`	`3972`	`// target can override this with custom lowering and calling the`
`3973`	`3973`	`// implementation functions.`
`3974`	`3974`	`LLT Ty = MRI.getType(MI.getOperand(0).getReg());`
`3975`		`- if (LI.isLegalOrCustom({G_UMIN, Ty}))`
	`3975`	`+ if (LI.isLegalOrCustom({G_UMIN, Ty}) && LI.isLegalOrCustom({G_UMAX, Ty}))`
`3976`	`3976`	`return lowerAddSubSatToMinMax(MI);`
`3977`	`3977`	`return lowerAddSubSatToAddoSubo(MI);`
`3978`	`3978`	`}`