[llvm] [AMDGPU] Let LowerModuleLDS run twice on the same module (PR #81729)
Jon Chesterfield via llvm-commits
llvm-commits at lists.llvm.org
Wed Feb 21 06:20:34 PST 2024
================
@@ -327,16 +327,18 @@ class AMDGPULowerModuleLDS {
return convertUsersOfConstantsToInstructions(LDSGlobals);
}
+ std::optional<bool> HasAbsoluteGVs;
----------------
JonChesterfield wrote:
This is only used by one function, make it a local variable and reinstate the static annotations. Minimising scope of mutable state is important.
https://github.com/llvm/llvm-project/pull/81729
More information about the llvm-commits
mailing list