[Openmp-commits] [PATCH] D91464: [OpenMP][OMPT] Implement verbose tool loading

[Hansang Bae via Phabricator via Openmp-commits]

hbae added a comment.

Can we also add print routine for OMP_TOOL_VERBOSE_INIT in kmp_settings.cpp (for OMP_DISPLAY_ENV) ?



================
Comment at: openmp/runtime/src/ompt-general.cpp:237
+  if (verbose_init)
+    fprintf(verbose_file,
+            "----- START LOGGING OF TOOL REGISTRATION -----\nSearch for OMP "
----------------
Can we put the common pattern `if (verbose_init) fprintf(verbose_file,` into a macro? e.g., `OMPT_VERBOSE(<message>)`. We may be able to add some common prefix to the macro if desired.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91464