[Openmp-commits] [PATCH] D123974: [OpenMP] [OMPT] [3/8] Implemented callback registration in libomptarget

[Dhruva Chakrabarti via Phabricator via Openmp-commits]

dhruvachak marked an inline comment as done.
dhruvachak added inline comments.


================
Comment at: openmp/libomptarget/include/ompt_device_callbacks.h:60
+/// Device callbacks object for the library that performs the instantiation
+extern OmptDeviceCallbacksTy ompt_device_callbacks;
+
----------------
jplehr wrote:
> I wonder if we should limit the use of snake_case to only the OMP/OMPT interface functions and types, and stick with the LLVM naming conventions otherwise?
Thanks for the comment. Changed.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123974