[llvm] [AMDGPU] Be less optimistic when allocating module scope lds (PR #161464)
Jon Chesterfield via llvm-commits
llvm-commits at lists.llvm.org
Wed Oct 1 06:39:43 PDT 2025
================
@@ -647,7 +649,8 @@ class AMDGPULowerModuleLDS {
ModuleScopeVariables.insert(GV);
} else if (K.second.size() == 1) {
KernelAccessVariables.insert(GV);
- } else if (set_is_subset(K.second, HybridModuleRootKernels)) {
+ } else if (K.second.size() == HybridModuleRootKernelsSize &&
----------------
JonChesterfield wrote:
That's the semantic intent certainly, it's looking for equality between the sets. I'll check what DenseSet::operator== does
https://github.com/llvm/llvm-project/pull/161464
More information about the llvm-commits
mailing list