Skip to content

Conversation

@cmc-rep
Copy link
Contributor

@cmc-rep cmc-rep commented Nov 6, 2024

No description provided.

@llvmbot
Copy link
Member

llvmbot commented Nov 6, 2024

@llvm/pr-subscribers-backend-amdgpu

Author: Gang Chen (cmc-rep)

Changes

Full diff: https://github.com/llvm/llvm-project/pull/115199.diff

1 Files Affected:

  • (modified) llvm/lib/Target/AMDGPU/AMDGPUMachineFunction.cpp (+1-1)
diff --git a/llvm/lib/Target/AMDGPU/AMDGPUMachineFunction.cpp b/llvm/lib/Target/AMDGPU/AMDGPUMachineFunction.cpp
index f52476464e05e6..ac11b11ef97ae5 100644
--- a/llvm/lib/Target/AMDGPU/AMDGPUMachineFunction.cpp
+++ b/llvm/lib/Target/AMDGPU/AMDGPUMachineFunction.cpp
@@ -103,7 +103,7 @@ unsigned AMDGPUMachineFunction::allocateLDSGlobal(const DataLayout &DL,
 
   unsigned Offset;
   if (GV.getAddressSpace() == AMDGPUAS::LOCAL_ADDRESS) {
-    if (TargetExtType *TTy = AMDGPU::isNamedBarrier(GV)) {
+    if (AMDGPU::isNamedBarrier(GV)) {
       std::optional<unsigned> BarAddr = getLDSAbsoluteAddress(GV);
       if (!BarAddr)
         llvm_unreachable("named barrier should have an assigned address");

@cmc-rep cmc-rep merged commit f85be26 into llvm:main Nov 6, 2024
8 of 9 checks passed
AdamGlass pushed a commit to AdamGlass/llvm-project that referenced this pull request Dec 30, 2024
AdamGlass pushed a commit to AdamGlass/llvm-project that referenced this pull request Dec 30, 2024
steven-studio pushed a commit to steven-studio/llvm-project that referenced this pull request Sep 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants