[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