[PATCH] D94823: AMDGPU: Add occupancy to serialized MachineFunctionInfo

[Matt Arsenault via Phabricator via llvm-commits]

arsenm closed this revision.
arsenm added a comment.

20566a2ed825c05d56708552d33d95ee12255f46 <https://reviews.llvm.org/rG20566a2ed825c05d56708552d33d95ee12255f46>



================
Comment at: llvm/lib/Target/AMDGPU/SIMachineFunctionInfo.h:278
 
+  // TODO: 10 may be a better default since it's the maximum.
+  unsigned Occupancy = 0;
----------------
rampitec wrote:
> Maximum is ST dependent. It can be better to use getMaxWavesPerEU().
We don't have access to that during construction though


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D94823/new/

https://reviews.llvm.org/D94823