[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