[Openmp-commits] [PATCH] D47717: [OMPT] Make sure that OMPT is enabled in runtime entry points that access internals of the runtime

John Mellor-Crummey via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Mon Jun 18 04:53:43 PDT 2018


jmellorcrummey requested changes to this revision.
jmellorcrummey added a comment.
This revision now requires changes to proceed.

None of this makes sense. If OMPT is not enabled, there should be no call to the ompt_initializer. For that reason, no tool should get pointers to OMPT runtime entry points if OMPT is not enabled. Therefore, the OMPT runtime entry points don’t need to check if OMPT is enabled.


https://reviews.llvm.org/D47717





More information about the Openmp-commits mailing list