[RISCV][NFC] Simplify ISD::SELECT Legality (#142650)

lenary · web-flow · commit 2863c640fa0d · 2025-06-03T12:49:42.000-07:00
ISD::SELECT is legal by default, so this change to the conditional makes
it clearer that XTHeadCondMov and XMipsCMove both leave this operation
legal rather than custom expanding it.
diff --git a/llvm/lib/Target/RISCV/RISCVISelLowering.cpp b/llvm/lib/Target/RISCV/RISCVISelLowering.cpp
@@ -434,9 +434,7 @@ RISCVTargetLowering::RISCVTargetLowering(const TargetMachine &TM,
     setOperationAction(ISD::ABS, MVT::i32, Custom);
   }
 
-  if (Subtarget.useCCMovInsn())
-    setOperationAction(ISD::SELECT, XLenVT, Legal);
-  else if (!Subtarget.hasVendorXTHeadCondMov())
+  if (!Subtarget.useCCMovInsn() && !Subtarget.hasVendorXTHeadCondMov())
     setOperationAction(ISD::SELECT, XLenVT, Custom);
 
   if (Subtarget.hasVendorXqcia() && !Subtarget.is64Bit()) {

Original file line number	Diff line number	Diff line change
`@@ -434,9 +434,7 @@ RISCVTargetLowering::RISCVTargetLowering(const TargetMachine &TM,`
`434`	`434`	`setOperationAction(ISD::ABS, MVT::i32, Custom);`
`435`	`435`	`}`
`436`	`436`
`437`		`- if (Subtarget.useCCMovInsn())`
`438`		`- setOperationAction(ISD::SELECT, XLenVT, Legal);`
`439`		`- else if (!Subtarget.hasVendorXTHeadCondMov())`
	`437`	`+ if (!Subtarget.useCCMovInsn() && !Subtarget.hasVendorXTHeadCondMov())`
`440`	`438`	`setOperationAction(ISD::SELECT, XLenVT, Custom);`
`441`	`439`
`442`	`440`	`if (Subtarget.hasVendorXqcia() && !Subtarget.is64Bit()) {`