[Openmp-commits] [PATCH] D145924: [OpenMP][libomptarget][AMDGPU] Update print launch info

[Johannes Doerfert via Phabricator via Openmp-commits]

jdoerfert added inline comments.


================
Comment at: openmp/libomptarget/plugins-nextgen/amdgpu/utils/UtilitiesRTL.h:204
+      assert(DN.isArray());
+      auto DNA = DN.getArray();
+
----------------
Can we assert that there are at most three? Also, add a message to each assert.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145924