[Openmp-commits] [PATCH] D123445: [OpenMP][FIX] Ensure to set the context for wait events if necessary

Johannes Doerfert via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Tue Apr 12 14:44:17 PDT 2022


This revision was automatically updated to reflect the committed changes.
Closed by commit rGa3a42c3ca23e: [OpenMP][FIX] Ensure to set the context for wait events if necessary (authored by jdoerfert).

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123445

Files:
  openmp/libomptarget/plugins/cuda/src/rtl.cpp


Index: openmp/libomptarget/plugins/cuda/src/rtl.cpp
===================================================================
--- openmp/libomptarget/plugins/cuda/src/rtl.cpp
+++ openmp/libomptarget/plugins/cuda/src/rtl.cpp
@@ -1734,7 +1734,10 @@
   assert(DeviceRTL.isValidDeviceId(device_id) && "device_id is invalid");
   assert(async_info_ptr && "async_info_ptr is nullptr");
   assert(event_ptr && "event is nullptr");
-  // NOTE: We might not need to set context for event sync.
+  // If we don't have a queue we need to set the context.
+  if (!async_info_ptr->Queue &&
+      DeviceRTL.setContext(device_id) != OFFLOAD_SUCCESS)
+    return OFFLOAD_FAIL;
   return DeviceRTL.waitEvent(device_id, async_info_ptr, event_ptr);
 }
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D123445.422337.patch
Type: text/x-patch
Size: 730 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/openmp-commits/attachments/20220412/a79e0161/attachment.bin>


More information about the Openmp-commits mailing list