[PATCH] D136103: OpenMP asynchronous memory copy support

Jisheng Zhao via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 24 19:45:56 PDT 2022


jz10 updated this revision to Diff 470357.
jz10 added a comment.

Thanks Shilei

1. "Does it work on AMDGPU and other targets? Why does it require Nvidia here?"

No, I just remove this line

2. "We don't have any performance test cases yet, and I'm not sure we need them right now."

Yes, I remove those two performance test cases


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

https://reviews.llvm.org/D136103

Files:
  openmp/libomptarget/include/interop.h
  openmp/libomptarget/src/api.cpp
  openmp/libomptarget/src/exports
  openmp/libomptarget/src/private.h
  openmp/libomptarget/test/api/omp_target_memcpy_async1.c
  openmp/libomptarget/test/api/omp_target_memcpy_async2.c
  openmp/libomptarget/test/api/omp_target_memcpy_rect_async1.c
  openmp/libomptarget/test/api/omp_target_memcpy_rect_async2.c

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D136103.470357.patch
Type: text/x-patch
Size: 21777 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20221025/73a89342/attachment-0001.bin>


More information about the cfe-commits mailing list