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

[Johannes Doerfert via Phabricator via Openmp-commits]

jdoerfert added inline comments.


================
Comment at: openmp/CMakeLists.txt:86
+option(OPENMP_ENABLE_LIBOMPTARGET_PROFILING "Enable time profiling for libomptarget."
+       ${ENABLE_LIBOMPTARGET})
 if (OPENMP_ENABLE_LIBOMPTARGET)
----------------
tianshilei1992 wrote:
> So is it intentional to enable profiling as long as `libomptarget` is enabled?
Yes, profiling is by default a single thread local memory access per entry point. If it turns out to be a problem we can revisit this.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93055