[PATCH] D80917: [OpenMP][AMDGCN] Support OpenMP offloading for AMDGCN architecture - Part 2

Saiyedul Islam via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 8 13:18:40 PDT 2020


saiislam marked 7 inline comments as done.
saiislam added inline comments.


================
Comment at: clang/include/clang/Basic/TargetInfo.h:202
+  const unsigned *GridValues;
+  const uint64_t *LongGridValues;
 
----------------
jdoerfert wrote:
> I would recommend adding a descriptive comment, even though that seems not to be the norm here.
> Why do we split these into "normal" and "long" grid values again?
> 
> `= nullptr` and an assert in the getters?
Realized no practical use of separate long values. Folded into regular enum and arrays.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80917





More information about the cfe-commits mailing list