This is an archive of the discontinued LLVM Phabricator instance.

[OpenMP][libomptarget] Fix AsyncInfoTy object in omp_target_memcpy
ClosedPublic

Authored by kevinsala on Nov 1 2022, 8:39 PM.

Details

Summary

The AsyncInfoTy should be created in the same device as the async operation will be issued. In omp_target_memcpy, the AsyncInfoTy for the host to destination device transfer was created referring to the source device.

Diff Detail

Event Timeline

kevinsala created this revision.Nov 1 2022, 8:39 PM
Herald added a project: Restricted Project. · View Herald TranscriptNov 1 2022, 8:39 PM
kevinsala requested review of this revision.Nov 1 2022, 8:39 PM
tianshilei1992 accepted this revision.Nov 1 2022, 9:19 PM

Thanks for the fix! LG.

This revision is now accepted and ready to land.Nov 1 2022, 9:19 PM

Thanks for reviewing it @tianshilei1992. Since I don't have access to the repo, I'd appreciate if you could commit the patch for me. Thanks!