[Openmp-commits] [PATCH] D93034: [OpenMP][OMPT] Fix OMPT address handling after D92197

[Joachim Protze via Phabricator via Openmp-commits]

protze.joachim created this revision.
protze.joachim added a reviewer: hbae.
protze.joachim added a project: OpenMP.
Herald added subscribers: guansong, yaxunl.
protze.joachim requested review of this revision.
Herald added a reviewer: jdoerfert.
Herald added a subscriber: sstefan1.

The split of interfaces in D92197 <https://reviews.llvm.org/D92197> breaks some of the OMPT tests by storing the wrong return or frame address.
This patch also introduces KMP_DEBUG_ASSERTIONS to verify expected state of the stored return address.
The return address is now managed in a thread-local stack in order to avoid repeated setting and unsetting the address.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D93034

Files:
  openmp/runtime/src/kmp_barrier.cpp
  openmp/runtime/src/kmp_cancel.cpp
  openmp/runtime/src/kmp_csupport.cpp
  openmp/runtime/src/kmp_dispatch.cpp
  openmp/runtime/src/kmp_gsupport.cpp
  openmp/runtime/src/kmp_runtime.cpp
  openmp/runtime/src/kmp_taskdeps.cpp
  openmp/runtime/src/kmp_tasking.cpp
  openmp/runtime/src/ompt-general.cpp
  openmp/runtime/src/ompt-internal.h
  openmp/runtime/src/ompt-specific.h
  openmp/runtime/test/ompt/cancel/cancel_parallel.c
  openmp/runtime/test/ompt/cancel/cancel_taskgroup.c

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D93034.310892.patch
Type: text/x-patch
Size: 48236 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/openmp-commits/attachments/20201210/52a2d781/attachment-0001.bin>