[Openmp-commits] [PATCH] D64442: [OMPT] Cleanup reset of exit_frame pointer

Jonas Hahnfeld via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Tue Jul 9 12:29:19 PDT 2019


Hahnfeld marked an inline comment as done.
Hahnfeld added a comment.

I also tried to test that the `exit_frame` is reset for serialized regions, but we don't have an implicit barrier then :-(



================
Comment at: openmp/runtime/src/kmp_runtime.cpp:1828
           if (ompt_enabled.enabled) {
-            exit_runtime_p = NULL;
             if (ompt_enabled.ompt_callback_implicit_task) {
----------------
I think this was actually wrong before, but I didn't manage to write a test for this code path.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D64442





More information about the Openmp-commits mailing list