[Openmp-commits] [PATCH] D86307: [OpenMP] Pack first-private arguments to improve efficiency of data transfer

[Shilei Tian via Phabricator via Openmp-commits]

tianshilei1992 created this revision.
tianshilei1992 added reviewers: ye-luo, jdoerfert, JonChesterfield.
Herald added subscribers: openmp-commits, guansong, yaxunl.
Herald added a project: OpenMP.
tianshilei1992 requested review of this revision.
Herald added a subscriber: sstefan1.

In this patch, we pack all small first-private arguments, allocate and
transfer them all at once to reduce the number of data transfer which is very
expensive. The "small" is defined by that the argument size is less than a
predefined value. Currently it is 1024. I'm not sure whether it is a good one,
and that is an open question. Another question is, do we need to make it
configurable via an environment variable?


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D86307

Files:
  openmp/libomptarget/src/omptarget.cpp
  openmp/libomptarget/test/mapping/private_mapping.c

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D86307.286850.patch
Type: text/x-patch
Size: 18785 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/openmp-commits/attachments/20200820/22408b33/attachment-0001.bin>