[llvm] [SelectionDAG] Remove `NoNaNsFPMath` uses (PR #183448)

via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 9 22:01:42 PDT 2026


================
@@ -15572,7 +15573,8 @@ SDValue SITargetLowering::performMinMaxCombine(SDNode *N,
        (VT == MVT::v2bf16 && Subtarget->hasBF16PackedInsts()) ||
        (VT == MVT::v2f16 && Subtarget->hasVOP3PInsts())) &&
       Op0.hasOneUse()) {
-    if (SDValue Res = performFPMed3ImmCombine(DAG, SDLoc(N), Op0, Op1))
+    if (SDValue Res = performFPMed3ImmCombine(DAG, SDLoc(N), Op0, Op1,
+                                              N->getFlags().hasNoNaNs()))
----------------
paperchalice wrote:

I'm conservative about propagating all flags, for `performFPMed3ImmCombine` case, `nnan` is enough and changing it to accept `SDNodeFlags` in future wouldn't be difficult...

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


More information about the llvm-commits mailing list