[llvm] AMDGPU: Fix gcc -Wenum-compare warning (PR #138529)
Matt Arsenault via llvm-commits
llvm-commits at lists.llvm.org
Mon May 5 06:17:07 PDT 2025
https://github.com/arsenm created https://github.com/llvm/llvm-project/pull/138529
It doesn't like this extended enum idiom, so make the code worse.
Fixes #125549
>From 1d6f62bbf658bf5349b8d922edb2809c6185ea17 Mon Sep 17 00:00:00 2001
From: Matt Arsenault <Matthew.Arsenault at amd.com>
Date: Mon, 5 May 2025 15:14:11 +0200
Subject: [PATCH] AMDGPU: Fix gcc -Wenum-compare warning
It doesn't like this extended enum idiom, so make the code worse.
Fixes #125549
---
llvm/lib/Target/AMDGPU/AMDGPUISelLowering.cpp | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/llvm/lib/Target/AMDGPU/AMDGPUISelLowering.cpp b/llvm/lib/Target/AMDGPU/AMDGPUISelLowering.cpp
index 236c373e70250..b6023b4f3fbcf 100644
--- a/llvm/lib/Target/AMDGPU/AMDGPUISelLowering.cpp
+++ b/llvm/lib/Target/AMDGPU/AMDGPUISelLowering.cpp
@@ -2971,7 +2971,8 @@ SDValue AMDGPUTargetLowering::lowerFEXP10Unsafe(SDValue X, const SDLoc &SL,
SelectionDAG &DAG,
SDNodeFlags Flags) const {
const EVT VT = X.getValueType();
- const unsigned Exp2Op = VT == MVT::f32 ? AMDGPUISD::EXP : ISD::FEXP2;
+ const unsigned Exp2Op = VT == MVT::f32 ? static_cast<unsigned>(AMDGPUISD::EXP)
+ : static_cast<unsigned>(ISD::FEXP2);
if (VT != MVT::f32 || !needsDenormHandlingF32(DAG, X, Flags)) {
// exp2(x * 0x1.a92000p+1f) * exp2(x * 0x1.4f0978p-11f);
More information about the llvm-commits
mailing list