[llvm] [AMDGPU] Max. WG size-induced occupancy limits max. waves/EU (PR #137807)
Matt Arsenault via llvm-commits
llvm-commits at lists.llvm.org
Thu May 1 03:14:53 PDT 2025
================
@@ -81,7 +81,7 @@ AMDGPUSubtarget::getOccupancyWithWorkGroupSizes(uint32_t LDSBytes,
// workgroups, maximum number of waves, and minimum occupancy. The opposite is
// generally true for the minimum group size. LDS or barrier ressource
// limitations can flip those minimums/maximums.
- const auto [MinWGSize, MaxWGSize] = getFlatWorkGroupSizes(F);
+ const auto &[MinWGSize, MaxWGSize] = FlatWorkGroupSizes;
----------------
arsenm wrote:
```suggestion
const auto [MinWGSize, MaxWGSize] = FlatWorkGroupSizes;
```
Don't see why this gained a reference
https://github.com/llvm/llvm-project/pull/137807
More information about the llvm-commits
mailing list