[PATCH] D65423: [AMDGPU] Fix high occupancy calculation and print it

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 29 16:09:15 PDT 2019


arsenm added inline comments.


================
Comment at: lib/Target/AMDGPU/AMDGPUSubtarget.h:1322
+
+  unsigned getMaxWavesPerEU() const override { return 10; }
 };
----------------
Should just set a field in the constructor rather than introducing a virtual 


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

https://reviews.llvm.org/D65423





More information about the llvm-commits mailing list