[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