[llvm] [AMDGPU] Inline mayIgnoreSignedZero into raw flag check (NFC) (PR #217580)
Arseniy Obolenskiy via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 20 03:30:54 PDT 2026
https://github.com/aobolensk created https://github.com/llvm/llvm-project/pull/217580
Precommit for https://github.com/llvm/llvm-project/pull/217573
>From dfd73fa0ce153ff777d5c666b43203b4181581fb Mon Sep 17 00:00:00 2001
From: Arseniy Obolenskiy <arseniy.obolenskiy at amd.com>
Date: Thu, 20 Aug 2026 12:30:18 +0200
Subject: [PATCH] [AMDGPU] Inline mayIgnoreSignedZero into raw flag check (NFC)
---
llvm/lib/Target/AMDGPU/AMDGPUISelLowering.cpp | 12 ++----------
llvm/lib/Target/AMDGPU/AMDGPUISelLowering.h | 2 --
2 files changed, 2 insertions(+), 12 deletions(-)
diff --git a/llvm/lib/Target/AMDGPU/AMDGPUISelLowering.cpp b/llvm/lib/Target/AMDGPU/AMDGPUISelLowering.cpp
index 3f93ca66dca17..7acc9ca2bafbd 100644
--- a/llvm/lib/Target/AMDGPU/AMDGPUISelLowering.cpp
+++ b/llvm/lib/Target/AMDGPU/AMDGPUISelLowering.cpp
@@ -648,14 +648,6 @@ AMDGPUTargetLowering::AMDGPUTargetLowering(const TargetMachine &TM,
setMaxLargeFPConvertBitWidthSupported(64);
}
-bool AMDGPUTargetLowering::mayIgnoreSignedZero(SDValue Op) const {
- const auto Flags = Op.getNode()->getFlags();
- if (Flags.hasNoSignedZeros())
- return true;
-
- return false;
-}
-
//===----------------------------------------------------------------------===//
// Target Information
//===----------------------------------------------------------------------===//
@@ -5230,7 +5222,7 @@ SDValue AMDGPUTargetLowering::performFNegCombine(SDNode *N,
SDLoc SL(N);
switch (Opc) {
case ISD::FADD: {
- if (!mayIgnoreSignedZero(N0) && !N->getFlags().hasNoSignedZeros())
+ if (!N0->getFlags().hasNoSignedZeros() && !N->getFlags().hasNoSignedZeros())
return SDValue();
// (fneg (fadd x, y)) -> (fadd (fneg x), (fneg y))
@@ -5278,7 +5270,7 @@ SDValue AMDGPUTargetLowering::performFNegCombine(SDNode *N,
case ISD::FMA:
case ISD::FMAD: {
// TODO: handle llvm.amdgcn.fma.legacy
- if (!mayIgnoreSignedZero(N0) && !N->getFlags().hasNoSignedZeros())
+ if (!N0->getFlags().hasNoSignedZeros() && !N->getFlags().hasNoSignedZeros())
return SDValue();
// (fneg (fma x, y, z)) -> (fma x, (fneg y), (fneg z))
diff --git a/llvm/lib/Target/AMDGPU/AMDGPUISelLowering.h b/llvm/lib/Target/AMDGPU/AMDGPUISelLowering.h
index f24f33e7dcb70..912315532b7d5 100644
--- a/llvm/lib/Target/AMDGPU/AMDGPUISelLowering.h
+++ b/llvm/lib/Target/AMDGPU/AMDGPUISelLowering.h
@@ -194,8 +194,6 @@ class AMDGPUTargetLowering : public TargetLowering {
AMDGPUTargetLowering(const TargetMachine &TM, const TargetSubtargetInfo &STI,
const AMDGPUSubtarget &AMDGPUSTI);
- bool mayIgnoreSignedZero(SDValue Op) const;
-
static inline SDValue stripBitcast(SDValue Val) {
return Val.getOpcode() == ISD::BITCAST ? Val.getOperand(0) : Val;
}
More information about the llvm-commits
mailing list