[llvm] [AMDGPU][SILoadStoreOptimizer] Fix unused variable warning (PR #177969)

Walter Lee via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 26 07:07:39 PST 2026


https://github.com/googlewalt updated https://github.com/llvm/llvm-project/pull/177969

>From aa4a6ca05eb8498dc56a6ed7c949b73aba83f024 Mon Sep 17 00:00:00 2001
From: Walter Lee <waltl at google.com>
Date: Mon, 26 Jan 2026 09:41:23 -0500
Subject: [PATCH 1/2] [AMDGPU][SILoadStoreOptimizer] Fix unused variable
 warning in SILoadStoreOptimizer.cpp

Fixes #176816 (13b20e7aeab83e82368be9ffd22ce02cb9b831ae).
---
 llvm/lib/Target/AMDGPU/SILoadStoreOptimizer.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/llvm/lib/Target/AMDGPU/SILoadStoreOptimizer.cpp b/llvm/lib/Target/AMDGPU/SILoadStoreOptimizer.cpp
index a3c5fb2fa6ca0..bbee86c603cd2 100644
--- a/llvm/lib/Target/AMDGPU/SILoadStoreOptimizer.cpp
+++ b/llvm/lib/Target/AMDGPU/SILoadStoreOptimizer.cpp
@@ -2282,8 +2282,8 @@ bool SILoadStoreOptimizer::processBaseWithConstOffset64(
   }
 
   // Now extract the base register (which should be a 64-bit VGPR).
-  MachineInstr *BaseDef = MRI->getVRegDef(BaseOp->getReg());
-  assert(BaseDef && "Expected definition for base register");
+  assert(MRI->getVRegDef(BaseOp->getReg()) &&
+         "Expected definition for base register");
   Addr.Base.LoReg = BaseOp->getReg();
   Addr.Base.UseV64Pattern = true;
   return true;

>From 56553f7343a80cadda512c847c2227e904d21591 Mon Sep 17 00:00:00 2001
From: Walter Lee <waltl at google.com>
Date: Mon, 26 Jan 2026 10:06:45 -0500
Subject: [PATCH 2/2] From review, can just delete the whole assert, this isn't
 querying the def anymore.

---
 llvm/lib/Target/AMDGPU/SILoadStoreOptimizer.cpp | 2 --
 1 file changed, 2 deletions(-)

diff --git a/llvm/lib/Target/AMDGPU/SILoadStoreOptimizer.cpp b/llvm/lib/Target/AMDGPU/SILoadStoreOptimizer.cpp
index bbee86c603cd2..594c35a1dee3b 100644
--- a/llvm/lib/Target/AMDGPU/SILoadStoreOptimizer.cpp
+++ b/llvm/lib/Target/AMDGPU/SILoadStoreOptimizer.cpp
@@ -2282,8 +2282,6 @@ bool SILoadStoreOptimizer::processBaseWithConstOffset64(
   }
 
   // Now extract the base register (which should be a 64-bit VGPR).
-  assert(MRI->getVRegDef(BaseOp->getReg()) &&
-         "Expected definition for base register");
   Addr.Base.LoReg = BaseOp->getReg();
   Addr.Base.UseV64Pattern = true;
   return true;



More information about the llvm-commits mailing list