[Openmp-commits] [openmp] 1081bb0 - [OpenMP][libomptarget] Fix run region async condition

Shilei Tian via Openmp-commits openmp-commits at lists.llvm.org
Mon Aug 15 10:08:48 PDT 2022


Author: Kevin Sala Penads
Date: 2022-08-15T13:08:45-04:00
New Revision: 1081bb08cc509af820849d7c9ef62d8e84bfcde2

URL: https://github.com/llvm/llvm-project/commit/1081bb08cc509af820849d7c9ef62d8e84bfcde2
DIFF: https://github.com/llvm/llvm-project/commit/1081bb08cc509af820849d7c9ef62d8e84bfcde2.diff

LOG: [OpenMP][libomptarget] Fix run region async condition

This patch fixes a condition in the openmp/libomptarget/src/device.cpp file. The code was checking if the run_region plugin API function was implemented, but it should actually check the run_region_async function instead.

Reviewed By: tianshilei1992

Differential Revision: https://reviews.llvm.org/D131782

Added: 
    

Modified: 
    openmp/libomptarget/src/device.cpp

Removed: 
    


################################################################################
diff  --git a/openmp/libomptarget/src/device.cpp b/openmp/libomptarget/src/device.cpp
index ddcbdc4a3932..29b5a6c7b9e6 100644
--- a/openmp/libomptarget/src/device.cpp
+++ b/openmp/libomptarget/src/device.cpp
@@ -580,7 +580,7 @@ int32_t DeviceTy::dataExchange(void *SrcPtr, DeviceTy &DstDev, void *DstPtr,
 int32_t DeviceTy::runRegion(void *TgtEntryPtr, void **TgtVarsPtr,
                             ptr
diff _t *TgtOffsets, int32_t TgtVarsSize,
                             AsyncInfoTy &AsyncInfo) {
-  if (!RTL->run_region || !RTL->synchronize)
+  if (!RTL->run_region_async || !RTL->synchronize)
     return RTL->run_region(RTLDeviceID, TgtEntryPtr, TgtVarsPtr, TgtOffsets,
                            TgtVarsSize);
   return RTL->run_region_async(RTLDeviceID, TgtEntryPtr, TgtVarsPtr, TgtOffsets,


        


More information about the Openmp-commits mailing list