[Openmp-commits] [PATCH] D124652: [OpenMP] [OMPT] [amdgpu] [5/8] Implemented device init/fini/load callbacks

Jan-Patrick Lehr via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Thu Dec 1 12:42:49 PST 2022


jplehr added inline comments.


================
Comment at: openmp/libomptarget/include/ompt_device_callbacks.h:53
+        ompt_callback_device_initialize_fn(
+            DeviceNum, Type, (ompt_device_t *)Device, doLookup, Documentation);
+      }
----------------
In other parts I saw that C-style casts were changed to static_cast or reinterpret_cast.
Potentially stick to these C++ casts instead is preferable.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124652



More information about the Openmp-commits mailing list