[Openmp-commits] [PATCH] D14027: [OMPT] Windows Support for OMPT

[Jonathan Peyton via Openmp-commits]

jlpeyton created this revision.
jlpeyton added reviewers: omalyshe, jmellorcrummey.
jlpeyton added a subscriber: openmp-commits.
jlpeyton set the repository for this revision to rL LLVM.

The problem is that the ompt_tool() function (which must be implemented by a performance tool) should be defined in the RTL as well to cover the case when the tool is not present in the address space of the process.   This functionality is accomplished with weak symbols in Unices.  Unfortunately, Windows does not support weak symbols.

The solution in these changes is to grab the list of all modules loaded by the process and then search for symbol "ompt_tool()" within them.  To avoid the case when ompt_tool() could be found in libomp.dll, a function with a different name is used -- ompt_aux_tool(). The right function is called by the pointer then.

While doing these changes, the OMPT_SUPPORT detection in CMake was changed to test for the required featuers for OMPT_SUPPORT, namely: __builtin_frame_address() existence, weak attribute existence and psapi.dll existence.  For LIBOMP_HAVE_OMPT_SUPPORT to be true, it must be that the __builtin_frame_address() intrinsic exists 
AND one of: either weak attributes exist or psapi.dll exists.

Also, since Process Status API is used I had to add new dependency -- psapi.dll to the library dependency micro test.

Repository:
  rL LLVM

http://reviews.llvm.org/D14027

Files:
  runtime/CMakeLists.txt
  runtime/cmake/LibompMicroTests.cmake
  runtime/cmake/config-ix.cmake
  runtime/src/kmp_config.h.cmake
  runtime/src/ompt-general.c
  runtime/src/ompt-specific.c

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D14027.38251.patch
Type: text/x-patch
Size: 7942 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/openmp-commits/attachments/20151023/7033b800/attachment.bin>