[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