[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