[llvm] SelectionDAG: Support FMINIMUMNUM and FMINIMUM in combineMinNumMaxNumImpl (PR #137449)

via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 28 08:21:51 PDT 2025


================
@@ -11516,9 +11516,18 @@ static SDValue combineMinNumMaxNumImpl(const SDLoc &DL, EVT VT, SDValue LHS,
   case ISD::SETLE:
   case ISD::SETULT:
   case ISD::SETULE: {
-    // Since it's known never nan to get here already, either fminnum or
-    // fminnum_ieee are OK. Try the ieee version first, since it's fminnum is
-    // expanded in terms of it.
+    // Since it's known never nan to get here already, either fminimumnum,
+    // fminimum, fminnum, or fminnum_ieee are OK. Try the ieee version first,
+    // since it's fminnum is expanded in terms of it.
----------------
LU-JOHN wrote:

Can you explain in the comment why IEEE2019Num and IEEE2019 are preferred in that order?

https://github.com/llvm/llvm-project/pull/137449


More information about the llvm-commits mailing list