[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