https://gcc.gnu.org/g:0b6d94ce72b2f35dbee7c42774d6972671c86f97
commit r15-4519-g0b6d94ce72b2f35dbee7c42774d6972671c86f97
Author: Andrew Stubbs <a...@baylibre.com>
Date: Mon Sep 16 12:31:59 2024 +0000
amdgcn: silence warning
FIRST_SGPR_REG is register zero so the compiler always claims this
comparison
is redundant. It's right, of course, but I'd have preferred to keep the
comparison for completeness. Probably the "correct" solution is to use an
enum
for these values.
gcc/ChangeLog:
* config/gcn/gcn.h (SGPR_REGNO_P): Silence warning.
Diff:
---
gcc/config/gcn/gcn.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/gcc/config/gcn/gcn.h b/gcc/config/gcn/gcn.h
index 1a4631dd39f6..faefe68cdfa9 100644
--- a/gcc/config/gcn/gcn.h
+++ b/gcc/config/gcn/gcn.h
@@ -191,7 +191,7 @@ STATIC_ASSERT (LAST_AVGPR_REG + 1 - FIRST_AVGPR_REG == 256);
#define HARD_FRAME_POINTER_IS_ARG_POINTER 0
#define HARD_FRAME_POINTER_IS_FRAME_POINTER 0
-#define SGPR_REGNO_P(N) ((N) >= FIRST_SGPR_REG && (N) <=
LAST_SGPR_REG)
+#define SGPR_REGNO_P(N) (/*(N) >= FIRST_SGPR_REG &&*/ (N) <=
LAST_SGPR_REG)
#define VGPR_REGNO_P(N) ((N) >= FIRST_VGPR_REG && (N) <=
LAST_VGPR_REG)
#define AVGPR_REGNO_P(N) ((N) >= FIRST_AVGPR_REG && (N) <=
LAST_AVGPR_REG)
#define SSRC_REGNO_P(N) ((N) <= SCC_REG && (N) != VCCZ_REG)
