[Openmp-commits] [PATCH] D41508: [OMPT] Build runtime with OMPT support by default

[Jonas Hahnfeld via Phabricator via Openmp-commits]

Hahnfeld accepted this revision.
Hahnfeld added reviewers: AndreyChurbanov, omalyshe, hbae.
Hahnfeld added a comment.
This revision is now accepted and ready to land.

LG. I'd feel better with an explicit acknowledgment from Intel though...

(We should probably add a few sentences to Clang's release notes so that users know the reason should they see a performance degradation. I can handle that after this patch landed.)



================
Comment at: README.rst:195-196
+**LIBOMP_OMPT_SUPPORT** = ``ON|OFF``
+  Include support for the OpenMP Tools Interface (OMPT). This option is ``OFF``
+  if this feature is not supported for this platform.
 
----------------
Should we mention which platforms are supported? Like `This option is ON by default for x86, x86_64, AArch64, and PPC64 on Linux, Windows, and mac OS`?


https://reviews.llvm.org/D41508