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

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 15 13:40:18 PST 2021


rampitec accepted this revision.
rampitec added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/lib/Target/AMDGPU/SIMachineFunctionInfo.h:278
 
+  // TODO: 10 may be a better default since it's the maximum.
+  unsigned Occupancy = 0;
----------------
Maximum is ST dependent. It can be better to use getMaxWavesPerEU().


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

https://reviews.llvm.org/D94823



More information about the llvm-commits mailing list