[llvm] [AMDGPU] Remove `NoNaNsFPMath` uses (PR #180469)
via llvm-commits
llvm-commits at lists.llvm.org
Sun Feb 8 21:45:39 PST 2026
https://github.com/paperchalice updated https://github.com/llvm/llvm-project/pull/180469
>From d36d9ceb7e758cfbea38cc745aa79dc94cc27ee7 Mon Sep 17 00:00:00 2001
From: PaperChalice <liujunchang97 at outlook.com>
Date: Mon, 9 Feb 2026 13:31:56 +0800
Subject: [PATCH] [AMDGPU] Remove `NoNaNsFPMath` uses
---
llvm/lib/Target/AMDGPU/AMDGPUISelLowering.cpp | 4 +---
llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp | 6 +-----
2 files changed, 2 insertions(+), 8 deletions(-)
diff --git a/llvm/lib/Target/AMDGPU/AMDGPUISelLowering.cpp b/llvm/lib/Target/AMDGPU/AMDGPUISelLowering.cpp
index 466cc6a5156d4..da21033388532 100644
--- a/llvm/lib/Target/AMDGPU/AMDGPUISelLowering.cpp
+++ b/llvm/lib/Target/AMDGPU/AMDGPUISelLowering.cpp
@@ -2776,7 +2776,6 @@ SDValue AMDGPUTargetLowering::LowerFLOGCommon(SDValue Op,
const bool IsLog10 = Op.getOpcode() == ISD::FLOG10;
assert(IsLog10 || Op.getOpcode() == ISD::FLOG);
- const auto &Options = getTargetMachine().Options;
if (VT == MVT::f16 || Flags.hasApproximateFuncs()) {
if (VT == MVT::f16 && !isTypeLegal(MVT::f16)) {
@@ -2845,8 +2844,7 @@ SDValue AMDGPUTargetLowering::LowerFLOGCommon(SDValue Op,
R = getMad(DAG, DL, VT, YH, CH, Mad1);
}
- const bool IsFiniteOnly =
- (Flags.hasNoNaNs() || Options.NoNaNsFPMath) && Flags.hasNoInfs();
+ const bool IsFiniteOnly = Flags.hasNoNaNs() && Flags.hasNoInfs();
// TODO: Check if known finite from source value.
if (!IsFiniteOnly) {
diff --git a/llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp b/llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp
index 6d4e4e64280c1..4a72e08c7b4e0 100644
--- a/llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp
+++ b/llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp
@@ -3559,9 +3559,6 @@ bool AMDGPULegalizerInfo::legalizeFlogCommon(MachineInstr &MI,
const LLT F32 = LLT::scalar(32);
const LLT F16 = LLT::scalar(16);
- const AMDGPUTargetMachine &TM =
- static_cast<const AMDGPUTargetMachine &>(MF.getTarget());
-
if (Ty == F16 || MI.getFlag(MachineInstr::FmAfn)) {
if (Ty == F16 && !ST.has16BitInsts()) {
Register LogVal = MRI.createGenericVirtualRegister(F32);
@@ -3630,8 +3627,7 @@ bool AMDGPULegalizerInfo::legalizeFlogCommon(MachineInstr &MI,
}
const bool IsFiniteOnly =
- (MI.getFlag(MachineInstr::FmNoNans) || TM.Options.NoNaNsFPMath) &&
- MI.getFlag(MachineInstr::FmNoInfs);
+ MI.getFlag(MachineInstr::FmNoNans) && MI.getFlag(MachineInstr::FmNoInfs);
if (!IsFiniteOnly) {
// Expand isfinite(x) => fabs(x) < inf
More information about the llvm-commits
mailing list