[Openmp-commits] [PATCH] D91692: [OpenMP][OMPT] Introduce a guard to handle OMPT return address

[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.

This is an alternative approach to address inconsistencies pointed out in: D90078 <https://reviews.llvm.org/D90078>
This patch makes sure that the return address is reset, when leaving the scope. 
In some cases, I had to move the macro out of an if-statement to have it in the right scope, in some cases I added an additional block to restrict the scope.

This patch does not handle inconsistencies, which might occur if the return address is still set when we call into the application.

In general, I would prefer to pass the address down the stack as function arguments, which is not possible with all the internal calls to external API functions


https://reviews.llvm.org/D91692

Files:
  openmp/runtime/src/kmp_csupport.cpp
  openmp/runtime/src/kmp_gsupport.cpp
  openmp/runtime/src/ompt-specific.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D91692.306012.patch
Type: text/x-patch
Size: 11119 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/openmp-commits/attachments/20201118/7be4c1dc/attachment-0001.bin>