[clang] [llvm] [AMDGPU] Add amdgpu-as MMRA for fences (PR #78572)
Pierre van Houtryve via llvm-commits
llvm-commits at lists.llvm.org
Tue May 21 23:00:12 PDT 2024
Pierre-vh wrote:
> > I thought image memory = private. It's unclear to me, what AS does OpenCL IMAGE memory map to in our backend? (But otherwise, yes, MMRA should just have the backend names, the mapping of the OpenCL IMAGE to a backend AS should be in the device-lib)
>
> Images are global memory with magical addressing and value interpretation on load/store. There's nothing private about them
So in our case:
- OpenCL Image = Global
- OpenCL Local = LDS
- OpenCL global = Global
?
Is that correct?
Then I guess the MMRA should just have "global" and "local" for now, we can always add more later if needed. What do you think?
https://github.com/llvm/llvm-project/pull/78572
More information about the llvm-commits
mailing list