[Openmp-commits] Patches to support OMPT

[Hahnfeld, Jonas]

Hi,

we don't have any commit rights at LLVM. It would be great if you could
commit the patches when all of them are ready.

Jonas

> -----Original Message-----
> From: Churbanov, Andrey [mailto:Andrey.Churbanov at intel.com]
> Sent: Monday, April 27, 2015 5:37 PM
> To: Hahnfeld, Jonas
> Cc: openmp-commits at dcs-maillist2.engr.illinois.edu; Cramer, Tim; John
> Mellor-Crummey (johnmc at rice.edu)
> Subject: RE: Patches to support OMPT
> 
> Jonas,
> 
> Just to clarify: do you expect we commit the patches, or do you have
enough
> rights to commit them yourself?
> 
> - Andrey
> 
> > -----Original Message-----
> > From: openmp-dev-bounces at cs.uiuc.edu [mailto:openmp-dev-
> > bounces at cs.uiuc.edu] On Behalf Of Hahnfeld, Jonas
> > Sent: Thursday, April 23, 2015 4:07 PM
> > To: 'openmp-dev at dcs-maillist2.engr.illinois.edu'
> > Subject: Re: [Openmp-dev] Patches to support OMPT
> >
> > Hi,
> >
> > we have just updated the revisions with your feedback.
> > Any further comments on this?
> >
> > Regards,
> > Jonas Hahnfeld
> >
> > > -----Original Message-----
> > > From: openmp-dev-bounces at cs.uiuc.edu [mailto:openmp-dev-
> > > bounces at cs.uiuc.edu] On Behalf Of Hahnfeld, Jonas
> > > Sent: Thursday, April 09, 2015 3:25 PM
> > > To: 'openmp-dev at dcs-maillist2.engr.illinois.edu'
> > > Cc: John Mellor-Crummey (johnmc at rice.edu); Cramer, Tim
> > > Subject: [Openmp-dev] Patches to support OMPT
> > >
> > > Hi all!
> > >
> > > After some months of work, we would like to share our patches to add
> > > support for OMPT[1] to this version of the Intel runtime.
> > > Implementation was mainly done by John Mellor-Crummey from Rice
> > > University in a repository at Google Code[2].
> > >
> > > Support for OMPT is disabled by default and therefore has to be
> > > enabled
> > > explicitly:
> > > With CMake, define "ompt-support": $ cmake -Dompt-support=true ..
> > > build.pl works nearly the same: $ make build_args="--ompt-support=on"
> > > With this switch, ompt-blame and ompt-trace will be enabled as well
> > > which can be disabled separately.
> > >
> > > We've uploaded the patches to LLVM's Phabricator. Some short
> explanation:
> > > 0001-kmp_lock-Change-return-type-of-release-functions-fro.patch[3]
> > > This is a preparation patch to get information about the actual
> > > release
> > status
> > > from nested locks.
> > >
> > > 0002-Add-new-files-for-OMPT.patch[4]
> > > This patch contains the new files for OMPT and the needed changes to
> > > the build infrastructure.
> > >
> > > 0003-Modify-runtime-to-call-correct-functions-for-OMPT.patch[5]
> > > These are the actual changes in the runtime to issue OMPT-related
> > functions.
> > > All of them are surrounded by #if OMPT_SUPPORT and can be disabled
> > > (which is the default).
> > >
> > > The implementation is tested with the OMPT-TestSuite[6] and conforms
> > > with the OpenMP's 2nd Technical Report[1].
> > > Additionally we ran the OpenMP Validation Suite contained in this
> > repository
> > > to make sure that nothing broke.
> > >
> > > Please review and commit if found suitable.
> > > We know that these are rather big patches (for a big feature), but
> > > we couldn't come up with a better splitting.
> > >
> > > From Aachen,
> > > Tim Cramer and Jonas Hahnfeld
> > >
> > > [1]: http://openmp.org/mp-documents/ompt-tr2.pdf
> > > [2]: https://code.google.com/p/ompt-intel-openmp/
> > > [3]: http://reviews.llvm.org/D8916
> > > [4]: http://reviews.llvm.org/D8917
> > > [5]: http://reviews.llvm.org/D8918
> > > [6]: https://code.google.com/p/ompt-test-suite/
> > >
> > >
> > > --
> > > Jonas Hahnfeld, MATSE-Auszubildender
> > >
> > > IT Center
> > > Group: High Performance Computing
> > > Division: Computational Science and Engineering RWTH Aachen
> > > University Seffenter Weg 23 D 52074  Aachen (Germany)
> > > Hahnfeld at itc.rwth-aachen.de www.itc.rwth-aachen.de
> > >
> 
> 
> --------------------------------------------------------------------
> Closed Joint Stock Company Intel A/O
> Registered legal address: Krylatsky Hills Business Park,
> 17 Krylatskaya Str., Bldg 4, Moscow 121614, Russian Federation
> 
> This e-mail and any attachments may contain confidential material for the
> sole use of the intended recipient(s). Any review or distribution by
others is
> strictly prohibited. If you are not the intended recipient, please contact
the
> sender and delete all copies.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 5868 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/openmp-commits/attachments/20150429/69953138/attachment.bin>