Skip to content

Commit 14a42e6

Browse files
authored
[AMDGPU] Remove NoInfsFPMath uses (llvm#163028)
Only `ninf` should be used.
1 parent 8a436b9 commit 14a42e6

File tree

2 files changed

+5
-8
lines changed

2 files changed

+5
-8
lines changed

llvm/lib/Target/AMDGPU/AMDGPUISelLowering.cpp

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2833,8 +2833,8 @@ SDValue AMDGPUTargetLowering::LowerFLOGCommon(SDValue Op,
28332833
R = getMad(DAG, DL, VT, YH, CH, Mad1);
28342834
}
28352835

2836-
const bool IsFiniteOnly = (Flags.hasNoNaNs() || Options.NoNaNsFPMath) &&
2837-
(Flags.hasNoInfs() || Options.NoInfsFPMath);
2836+
const bool IsFiniteOnly =
2837+
(Flags.hasNoNaNs() || Options.NoNaNsFPMath) && Flags.hasNoInfs();
28382838

28392839
// TODO: Check if known finite from source value.
28402840
if (!IsFiniteOnly) {
@@ -3161,9 +3161,8 @@ SDValue AMDGPUTargetLowering::lowerFEXP(SDValue Op, SelectionDAG &DAG) const {
31613161
DAG.getSetCC(SL, SetCCVT, X, UnderflowCheckConst, ISD::SETOLT);
31623162

31633163
R = DAG.getNode(ISD::SELECT, SL, VT, Underflow, Zero, R);
3164-
const auto &Options = getTargetMachine().Options;
31653164

3166-
if (!Flags.hasNoInfs() && !Options.NoInfsFPMath) {
3165+
if (!Flags.hasNoInfs()) {
31673166
SDValue OverflowCheckConst =
31683167
DAG.getConstantFP(IsExp10 ? 0x1.344136p+5f : 0x1.62e430p+6f, SL, VT);
31693168
SDValue Overflow =

llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3575,7 +3575,7 @@ bool AMDGPULegalizerInfo::legalizeFlogCommon(MachineInstr &MI,
35753575

35763576
const bool IsFiniteOnly =
35773577
(MI.getFlag(MachineInstr::FmNoNans) || TM.Options.NoNaNsFPMath) &&
3578-
(MI.getFlag(MachineInstr::FmNoInfs) || TM.Options.NoInfsFPMath);
3578+
MI.getFlag(MachineInstr::FmNoInfs);
35793579

35803580
if (!IsFiniteOnly) {
35813581
// Expand isfinite(x) => fabs(x) < inf
@@ -3864,9 +3864,7 @@ bool AMDGPULegalizerInfo::legalizeFExp(MachineInstr &MI,
38643864

38653865
R = B.buildSelect(Ty, Underflow, Zero, R);
38663866

3867-
const auto &Options = MF.getTarget().Options;
3868-
3869-
if (!(Flags & MachineInstr::FmNoInfs) && !Options.NoInfsFPMath) {
3867+
if (!(Flags & MachineInstr::FmNoInfs)) {
38703868
auto OverflowCheckConst =
38713869
B.buildFConstant(Ty, IsExp10 ? 0x1.344136p+5f : 0x1.62e430p+6f);
38723870

0 commit comments

Comments
 (0)