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

Thu Aug 20 12:12:42 PDT 2020

ye-luo added a comment.

In D86307#2229059 <https://reviews.llvm.org/D86307#2229059>, @jdoerfert wrote:

> In D86307#2229032 <https://reviews.llvm.org/D86307#2229032>, @ye-luo wrote:
>
>> Why just "small" ones? why not all of them?
>
> In addition to the last paragraph of the new commit message, we also have to copy the data on the host in the right place. That is not free as the size grows.

Copy cost on the host is a convincing argument but the asynchronous benefit doesn't seem to me true.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D86307/new/

https://reviews.llvm.org/D86307