[Openmp-commits] [PATCH] D153837: Synchronize after each GPU action in the nextgen plugin

Johannes Doerfert via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Fri Jul 14 11:20:47 PDT 2023


jdoerfert added inline comments.


================
Comment at: openmp/libomptarget/plugins-nextgen/common/PluginInterface/PluginInterface.cpp:215
+  if (ForceSynchronize && !Err)
+    Err = Device.synchronize(AsyncInfoPtr);
+
----------------
Are we sure that this has the desired effect? Also, do we need to check for the presence of Queue?


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

https://reviews.llvm.org/D153837



More information about the Openmp-commits mailing list