[Openmp-dev] Fwd: [PATCH] D45890: [OMPT] Add implementation and tests of Archer tool

[Joachim Protze via Openmp-dev]

Hi all,

I propose to add a new directory at root level of the OpenMP repository 
to collect OMPT-based tools. This directory can also be used to checkout 
other tools into the directory and build them with access to the 
OpenMP-library configuration.

As a first tool I uploaded Archer to phabricator (to see how this would 
look like just follow the link at the end of this mail). This tool 
closes the gap between OpenMP runtime and ThreadSanitizer. At some point 
this will also supersede the TSan-annotations in the runtime code.

Another tool I would see is a tool to implement the omp-display-affinity 
functionality. We already have a prototype of that tool, and it should 
be quite easy to update this prototype to implement the latest spec.


Here I would like to get your feedback on:

- do you agree with the plan to integrate tools into the runtime repository?
- what licenses are acceptable for tools which want to get integrated?

Thanks,
Joachim

-------- Forwarded Message --------
Subject: [PATCH] D45890: [OMPT] Add implementation and tests of Archer tool
Date: Fri, 20 Apr 2018 17:01:42 +0000 (UTC)
From: Joachim Protze via Phabricator <reviews at reviews.llvm.org>
Reply-To: reviews+D45890+public+16093abf34ab836a at reviews.llvm.org
To: protze.joachim at gmail.com, hfinkel at anl.gov, hahnjo at hahnjo.de
CC: mgorny at gentoo.org, openmp-commits at lists.llvm.org, 
filcab+llvm.phabricator at gmail.com, junbuml at codeaurora.org

protze.joachim created this revision.
protze.joachim added reviewers: hfinkel, Hahnfeld.
Herald added a subscriber: mgorny.

The tool provides TSAN annotations for OpenMP synchronization. Enabled 
for application execution by setting OMP_TOOL_LIBRARIES=libarcher.so, this
library should get installed in the same order as libomp, so the linker
should find the file when the application is linked with OpenMP.

The tool detects whether the application was built with TSan and rejects
activation according to the OMPT protocol if there is no TSan-rt.


Further it is recommended to set
TSAN_OPTIONS="ignore_noninstrumented_modules=1"
This avoids some false positive errors from the OpenMP runtime.


https://reviews.llvm.org/D45890