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

via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 27 04:05:53 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,
----------------
abhishek-kaushik22 wrote:

I would like to work on this, should I close this PR and raise a new issue?

Also what do you mean by 

> Maybe handle ordered if fabs is free?

in the `TODO`. 

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


More information about the llvm-commits mailing list