Skip to content

Commit 06f867c

Browse files
committed
expandFMINIMUMNUM_FMAXIMUMNUM: Quiet is not needed for NaN vs NaN
New LangRef doesn't requires quieting for NaN vs NaN, aka the result may be sNaN for sNaN vs NaN. See: #139228
1 parent d915355 commit 06f867c

File tree

1 file changed

+0
-5
lines changed

1 file changed

+0
-5
lines changed

llvm/lib/CodeGen/SelectionDAG/TargetLowering.cpp

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -8683,11 +8683,6 @@ SDValue TargetLowering::expandFMINIMUMNUM_FMAXIMUMNUM(SDNode *Node,
86838683

86848684
SDValue MinMax =
86858685
DAG.getSelectCC(DL, LHS, RHS, LHS, RHS, IsMax ? ISD::SETGT : ISD::SETLT);
8686-
// If MinMax is NaN, let's quiet it.
8687-
if (!Flags.hasNoNaNs() && !DAG.isKnownNeverNaN(LHS) &&
8688-
!DAG.isKnownNeverNaN(RHS)) {
8689-
MinMax = DAG.getNode(ISD::FCANONICALIZE, DL, VT, MinMax, Flags);
8690-
}
86918686

86928687
// Fixup signed zero behavior.
86938688
if (Options.NoSignedZerosFPMath || Flags.hasNoSignedZeros() ||

0 commit comments

Comments
 (0)