This is an archive of the discontinued LLVM Phabricator instance.

[OpenMP] Add an information flag for device data transfers
ClosedPublic

Authored by jhuber6 on Jun 8 2021, 1:46 PM.

Details

Summary

This patch adds an information flag that indicated when data is being copied to
and from the device. This will be helpful for finding redundant or unnecessary
data transfers in applications.

Diff Detail

Event Timeline

jhuber6 created this revision.Jun 8 2021, 1:46 PM
jhuber6 requested review of this revision.Jun 8 2021, 1:46 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 8 2021, 1:46 PM
jdoerfert accepted this revision.Jun 8 2021, 2:13 PM

LG, one nit

openmp/libomptarget/test/offloading/info.c
17–18

regexp probably.

This revision is now accepted and ready to land.Jun 8 2021, 2:13 PM
jhuber6 updated this revision to Diff 350713.Jun 8 2021, 2:16 PM

Fixing nits.

grokos added a subscriber: grokos.Jun 8 2021, 2:41 PM
grokos added inline comments.
openmp/docs/design/Runtimes.rst
92

coppies --> copied

openmp/libomptarget/include/Debug.h
55

transferred

openmp/libomptarget/src/device.cpp
428

Better use the PRId64 macro to print an int64_t. long int is not always equal to int64_t so on some systems this will trigger compilation warnings.

Same for retrieveData.

jhuber6 updated this revision to Diff 350735.Jun 8 2021, 4:26 PM

Fixing test and addressing comments.

grokos accepted this revision.Jun 8 2021, 4:47 PM
This revision was automatically updated to reflect the committed changes.