[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