[clang] [llvm] [AMDGPU] Add amdgpu-as MMRA for fences (PR #78572)
    Sameer Sahasrabuddhe via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Sun May  5 21:39:38 PDT 2024
    
    
  
================
@@ -18365,6 +18366,28 @@ Value *CodeGenFunction::EmitHLSLBuiltinExpr(unsigned BuiltinID,
   return nullptr;
 }
 
+void CodeGenFunction::AddAMDGCNFenceAddressSpaceMMRA(llvm::Instruction *Inst,
----------------
ssahasra wrote:
The function immediately below this uses "AMDGPU" in its name. I think that's the newer practice. I don't have a strong opinion on this, because the same file also has functions which say "AMDGCN" instead. I am not sure which way the naming convention is leaning.
https://github.com/llvm/llvm-project/pull/78572
    
    
More information about the llvm-commits
mailing list