[Openmp-commits] [PATCH] D93055: [OpenMP] Add time profiling for libomptarget

[Giorgis Georgakoudis via Phabricator via Openmp-commits]

ggeorgakoudis marked 5 inline comments as done.
ggeorgakoudis added inline comments.


================
Comment at: openmp/libomptarget/src/interface.cpp:136
                                            void **arg_mappers) {
+  TIMESCOPE
   if (IsOffloadDisabled()) return;
----------------
jdoerfert wrote:
> We should pass `loc` to `TIMESCOPE`.
> If it is not null, we can pass `reinterpret_cast<const char *>(loc->psource)` for now as second argument to the TimeScope constructor.
Left that for future work


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93055