[PATCH] D139467: AMDGPU: Add AMDGPUSubtarget::getEUsPerCU()

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 7 00:11:20 PST 2022


foad added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUSubtarget.h:213
 
+  // Number of SIMDs per "CU", where the "CU" is the unit onto which workgroups
+  // are mapped. This takes WGP mode vs. CU mode into account.
----------------
Since "CU" is misleading, can you think of a better name for "unit onto which workgroups are mapped"?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139467



More information about the llvm-commits mailing list