[llvm] [AMDGPU] Make readonly noilas kernel arguments constant memory (PR #139629)
Stanislav Mekhanoshin via llvm-commits
llvm-commits at lists.llvm.org
Mon May 12 14:57:32 PDT 2025
rampitec wrote:
> I thought we did this in the distant past but it was incorrect.
>
> Code elsewhere, in another running kernel or the host could still be modifying the memory. The current langref doesn't have any explicit wording on access from other threads
Yes, const only guaranties it is not modified in this function. Arguably one should use atomics for cross-thread communication though.
https://github.com/llvm/llvm-project/pull/139629
More information about the llvm-commits
mailing list