[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