[llvm] [AMDGPU] Occupancy w.r.t. workgroup size range is also a range (PR #123748)
Matt Arsenault via llvm-commits
llvm-commits at lists.llvm.org
Tue Jan 21 20:58:35 PST 2025
arsenm wrote:
> One potential issue I see is, the computation with workgroup size completely ignores `amdgpu-waves-per-eu` attribute, while it is considered with SGPRs and VGPRs. I understand the current implementation doesn't factor it in as well, but I don't know if we should.
As it should. The group size is a hard ABI requirement, and explicit. The amdgpu-waves-per-eu is an optimization hint and should not constrain the group size.
https://github.com/llvm/llvm-project/pull/123748
More information about the llvm-commits
mailing list