This is an archive of the discontinued LLVM Phabricator instance.

[OpenMP] [OMPT] [1/8] Create separate categories for host, device, [no]emi events
ClosedPublic

Authored by dhruvachak on Apr 8 2022, 6:29 PM.

Details

Summary

In preparation for OMPT target changes, create separate categories of events that will be used by OMPT target support.

Split up existing macro FOREACH_OMPT_EVENT into new ones. There is no change to the original macro. Created new macros FOREACH_OMPT_HOST_EVENT, FOREACH_OMPT_DEVICE_EVENT, FOREACH_OMPT_NOEMI_EVENT, FOREACH_OMPT_EMI_EVENT, and a few other sub-categories that can be used as required. One such use is in D123974 which uses events selectively.

Patch from John Mellor-Crummey <johnmc@rice.edu>

Diff Detail

Event Timeline

dhruvachak created this revision.Apr 8 2022, 6:29 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 8 2022, 6:29 PM
dhruvachak requested review of this revision.Apr 8 2022, 6:29 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 8 2022, 6:29 PM

This patch is split out of https://reviews.llvm.org/D113728 which shows where the new event categories are used.

dhruvachak retitled this revision from [OpenMP] Create separate categories for host, device, [no]emi events to [OpenMP] [OMPT] [1/8] Create separate categories for host, device, [no]emi events .Jun 20 2022, 6:01 PM
dhruvachak edited the summary of this revision. (Show Details)

@protze.joachim @dreachem Can you please check whether the new macros in this patch are correctly categorized? Do you see any problem with this change?

dreachem accepted this revision.Sep 7 2022, 8:01 AM

LGTM.

This revision is now accepted and ready to land.Sep 7 2022, 8:01 AM
dhruvachak retitled this revision from [OpenMP] [OMPT] [1/8] Create separate categories for host, device, [no]emi events to [OpenMP] [OMPT] [1/8] Create separate categories for host, device, [no]emi events.

Rebase