[clang] [llvm] [AMDGPU] Add initial support for VGPR as memory (PR #205435)

Diana Picus via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 3 05:33:35 PDT 2026


================
@@ -2275,6 +2288,10 @@ void AMDGPUCodeGenPassBuilder::addIRPasses(PassManagerWrapper &PMW) const {
   if (EnableLowerModuleLDS)
     addModulePass(AMDGPULowerModuleLDSPass(TM), PMW);
 
+  // Lay out "VGPR as memory" (addrspace(13)) globals into a shared register
+  // file (see the legacy pipeline above for details).
----------------
rovka wrote:

I'd say this is the other way around - you should add the comments here and either duplicate or reference to this from the legacy infra (since in a distant future that might get removed and people might not notice and update this comment in the process).

https://github.com/llvm/llvm-project/pull/205435


More information about the cfe-commits mailing list