[llvm] [AMDGPU] Common up two local memory size calculations. NFCI. (PR #154784)
Jay Foad via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 21 08:17:39 PDT 2025
https://github.com/jayfoad created https://github.com/llvm/llvm-project/pull/154784
None
>From 4807a7a3c1c1367093675248a19668fd99bd2f2a Mon Sep 17 00:00:00 2001
From: Jay Foad <jay.foad at amd.com>
Date: Thu, 21 Aug 2025 16:16:25 +0100
Subject: [PATCH] [AMDGPU] Common up two local memory size calculations. NFCI.
---
llvm/lib/Target/AMDGPU/GCNSubtarget.cpp | 5 +----
llvm/lib/Target/AMDGPU/Utils/AMDGPUBaseInfo.cpp | 2 +-
2 files changed, 2 insertions(+), 5 deletions(-)
diff --git a/llvm/lib/Target/AMDGPU/GCNSubtarget.cpp b/llvm/lib/Target/AMDGPU/GCNSubtarget.cpp
index fe9eeb9a68469..931966b6df1df 100644
--- a/llvm/lib/Target/AMDGPU/GCNSubtarget.cpp
+++ b/llvm/lib/Target/AMDGPU/GCNSubtarget.cpp
@@ -140,10 +140,7 @@ GCNSubtarget &GCNSubtarget::initializeSubtargetDependencies(const Triple &TT,
if (AddressableLocalMemorySize == 0)
AddressableLocalMemorySize = 32768;
- LocalMemorySize = AddressableLocalMemorySize;
- if (AMDGPU::isGFX10Plus(*this) &&
- !getFeatureBits().test(AMDGPU::FeatureCuMode))
- LocalMemorySize *= 2;
+ LocalMemorySize = AMDGPU::IsaInfo::getLocalMemorySize(this);
HasFminFmaxLegacy = getGeneration() < AMDGPUSubtarget::VOLCANIC_ISLANDS;
HasSMulHi = getGeneration() >= AMDGPUSubtarget::GFX9;
diff --git a/llvm/lib/Target/AMDGPU/Utils/AMDGPUBaseInfo.cpp b/llvm/lib/Target/AMDGPU/Utils/AMDGPUBaseInfo.cpp
index 6e4e087ed276e..18ee9c16b3ff9 100644
--- a/llvm/lib/Target/AMDGPU/Utils/AMDGPUBaseInfo.cpp
+++ b/llvm/lib/Target/AMDGPU/Utils/AMDGPUBaseInfo.cpp
@@ -1163,7 +1163,7 @@ unsigned getAddressableLocalMemorySize(const MCSubtargetInfo *STI) {
return 163840;
if (STI->getFeatureBits().test(FeatureAddressableLocalMemorySize327680))
return 327680;
- return 0;
+ return 32768;
}
unsigned getEUsPerCU(const MCSubtargetInfo *STI) {
More information about the llvm-commits
mailing list