Skip to content

Commit a177172

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: llvm#139228
1 parent b8ba963 commit a177172

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
@@ -8684,11 +8684,6 @@ SDValue TargetLowering::expandFMINIMUMNUM_FMAXIMUMNUM(SDNode *Node,
86848684

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

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

0 commit comments

Comments
 (0)