[PATCH] D154207: [AMDGPU] Rename predefined macro __AMDGCN_WAVEFRONT_SIZE

Matt Arsenault via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jun 30 06:18:41 PDT 2023


arsenm added inline comments.


================
Comment at: clang/lib/Basic/Targets/AMDGPU.cpp:318
 
+  Builder.defineMacro("__AMDGCN_WAVEFRONT_SIZE__", Twine(WavefrontSize));
+  // ToDo: deprecate this macro for naming consistency.
----------------
If you're renaming it anyway, might as well go for __AMDGPU?


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

https://reviews.llvm.org/D154207



More information about the cfe-commits mailing list