[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