This is an archive of the discontinued LLVM Phabricator instance.

[OMPT] Provide the right thread_num for ancestor levels
ClosedPublic

Authored by protze.joachim on May 7 2018, 9:42 AM.

Details

Summary

The current implementation always provides the thread-num for the current parallel region. This patch fixes the behavior for ancestor levels >0.

Diff Detail

Repository
rL LLVM

Event Timeline

protze.joachim created this revision.May 7 2018, 9:42 AM
Hahnfeld accepted this revision.May 8 2018, 12:22 AM

LGTM with some minor comments.

runtime/src/ompt-specific.cpp
411 ↗(On Diff #145485)

Please remove.

runtime/test/ompt/parallel/nested_thread_num.c
1 ↗(On Diff #145485)

I think the CHECK prefix doesn't add additional coverage, can we remove this run?

This revision is now accepted and ready to land.May 8 2018, 12:22 AM
This revision was automatically updated to reflect the committed changes.