This is an archive of the discontinued LLVM Phabricator instance.

[OpenMP] Begin Adding OpenMP Tool to Gather OpenMP Information
ClosedPublic

Authored by jhuber6 on Nov 9 2020, 8:08 AM.

Details

Summary

This begins to add support for a set of scripts that can be used to get information from OpenMP programs to better describe problems and eventually show the data to the user in formatted output. Right now the only support is forformatting the register and memory usage reports from ptxas and nvlink. This is simply done as a wrapper around clang and clang++.

Diff Detail

Event Timeline

jhuber6 created this revision.Nov 9 2020, 8:08 AM
Herald added a project: Restricted Project. · View Herald TranscriptNov 9 2020, 8:08 AM
jhuber6 requested review of this revision.Nov 9 2020, 8:08 AM

Do we need the ++ version, it looks the same.

Can we have a test for this in the OpenMP runtime test folder maybe?

jhuber6 updated this revision to Diff 304628.Nov 11 2020, 1:15 PM

Updating to remove some redundancy. The ++ variant is simply a link but it appears like that doesn't appear in the diff viewer.

jhuber6 updated this revision to Diff 304642.Nov 11 2020, 1:40 PM

Changing hard link to a symbolic link.

jdoerfert accepted this revision.Nov 11 2020, 4:21 PM

I think this is a good start, we can improve on it as we go. Please check if we need license information in the python files, e.g., check other python files. Also, one nit below.

openmp/tools/analyzer/analyzer.py
28

Do we require _Z (here and above), isn't that only present for C++ ?

This revision is now accepted and ready to land.Nov 11 2020, 4:21 PM
jhuber6 updated this revision to Diff 304684.Nov 11 2020, 4:56 PM

Fixing regex for C function names.

This revision was landed with ongoing or failed builds.Nov 11 2020, 5:01 PM
This revision was automatically updated to reflect the committed changes.