[Openmp-commits] [PATCH] D96067: [OpenMP][libomptarget] Fixed an issue that device sync is skipped if the kernel doesn't have any argument

[Johannes Doerfert via Phabricator via Openmp-commits]

jdoerfert accepted this revision.
jdoerfert added a comment.
This revision is now accepted and ready to land.

LGTM, one NIT below.



================
Comment at: openmp/libomptarget/src/omptarget.cpp:1323
+    // this point.
+    return syncDevice(Device, &AsyncInfo);
   }
----------------
", and it is invoked asynchronously", really? Maybe the language is just confusing.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96067