[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