================
@@ -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
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to