Revert "[OpenMP] Handle function calls from libomp to `libomptarget…

…` correctly" This reverts commit 479e335. The assertion at `kmp_tasking.cpp(29)` is triggered.
llvm · May 1, 2023 · 284e54d · 284e54d
1 parent 479e335
commit 284e54d
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 26 deletions.
diff --git a/openmp/runtime/src/kmp.h b/openmp/runtime/src/kmp.h
@@ -1100,12 +1100,6 @@ extern void __kmp_init_target_mem();
 
 /* ------------------------------------------------------------------------ */
 
-#if ENABLE_LIBOMPTARGET
-extern void __kmp_init_target_task();
-#endif
-
-/* ------------------------------------------------------------------------ */
-
 #define KMP_UINT64_MAX                                                         \
   (~((kmp_uint64)1 << ((sizeof(kmp_uint64) * (1 << 3)) - 1)))
 

diff --git a/openmp/runtime/src/kmp_runtime.cpp b/openmp/runtime/src/kmp_runtime.cpp
@@ -553,14 +553,6 @@ static void __kmp_fini_allocator() { __kmp_fini_memkind(); }
 
 /* ------------------------------------------------------------------------ */
 
-#if ENABLE_LIBOMPTARGET
-static void __kmp_init_omptarget() {
-  __kmp_init_target_task();
-}
-#endif
-
-/* ------------------------------------------------------------------------ */
-
 #if KMP_DYNAMIC_LIB
 #if KMP_OS_WINDOWS
 
@@ -7049,11 +7041,6 @@ static void __kmp_do_serial_initialize(void) {
 
   __kmp_validate_locks();
 
-#if ENABLE_LIBOMPTARGET
-  /* Initialize functions from libomptarget */
-  __kmp_init_omptarget();
-#endif
-
   /* Initialize internal memory allocator */
   __kmp_init_allocator();
 

diff --git a/openmp/runtime/src/kmp_tasking.cpp b/openmp/runtime/src/kmp_tasking.cpp
@@ -22,12 +22,8 @@
 #endif
 
 #if ENABLE_LIBOMPTARGET
-static void (*tgt_target_nowait_query)(void **);
-
-void __kmp_init_target_task() {
-  *(void **)(&tgt_target_nowait_query) = KMP_DLSYM("__tgt_target_nowait_query");
-  KMP_ASSERT(tgt_target_nowait_query);
-}
+// Declaration of synchronization function from libomptarget.
+extern "C" void __tgt_target_nowait_query(void **) KMP_WEAK_ATTRIBUTE_INTERNAL;
 #endif
 
 /* forward declaration */
@@ -1809,7 +1805,7 @@ static void __kmp_invoke_task(kmp_int32 gtid, kmp_task_t *task,
       // If we have a valid target async handle, that means that we have already
       // executed the task routine once. We must query for the handle completion
       // instead of re-executing the routine.
-      tgt_target_nowait_query(&taskdata->td_target_data.async_handle);
+      __tgt_target_nowait_query(&taskdata->td_target_data.async_handle);
     } else
 #endif
     if (task->routine != NULL) {