[llvm] DAG: Skip 0 sign handling in minimum/maximum lowering for _ieee case (PR #91326)
Min-Yih Hsu via llvm-commits
llvm-commits at lists.llvm.org
Wed May 8 09:32:24 PDT 2024
================
@@ -8400,8 +8400,14 @@ SDValue TargetLowering::expandFMINIMUM_FMAXIMUM(SDNode *N,
SDValue MinMax;
unsigned CompOpcIeee = IsMax ? ISD::FMAXNUM_IEEE : ISD::FMINNUM_IEEE;
unsigned CompOpc = IsMax ? ISD::FMAXNUM : ISD::FMINNUM;
+
+ // FIXME: We should probably define fminnum/fmaxnum variants with correct
----------------
mshockwave wrote:
what do you mean by "correct" here? Something like consolidating FMAX/MINNUM with FMAX/MINNUM_IEEE ?
https://github.com/llvm/llvm-project/pull/91326
More information about the llvm-commits
mailing list