[llvm] Remove redundant ternary operator in conditional check (PR #105629)

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 22 04:42:26 PDT 2024


================
@@ -8755,7 +8755,7 @@ SDValue TargetLowering::expandIS_FPCLASS(EVT ResultVT, SDValue Op,
       ISD::CondCode OrderedOp = IsInverted ? ISD::SETUGE : ISD::SETOLT;
       ISD::CondCode UnorderedOp = IsInverted ? ISD::SETOGE : ISD::SETULT;
 
-      if (isCondCodeLegalOrCustom(IsOrdered ? OrderedOp : UnorderedOp,
+      if (isCondCodeLegalOrCustom(UnorderedOp,
----------------
arsenm wrote:

Considering the todo above, maybe it's worth trying to remove the IsOrdered check and seeing if there's a way to improve the x86 output? 

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


More information about the llvm-commits mailing list