more comments

Author: Alexander Batashev

sycl/cmake/modules/AddSYCL.cmake

@@ -33,19 +33,21 @@ endfunction()
 function(add_sycl_plugin PLUGIN_NAME)
   cmake_parse_arguments("ARG"
     ""
-    "LINKER_SCRIPT"
+    ""
     "SOURCES;INCLUDE_DIRS;LIBRARIES"
     ${ARGN}
   )
 
   add_sycl_library("pi_${PLUGIN_NAME}" SHARED
     TOOLCHAIN
-    LINKER_SCRIPT "${sycl_src_dir}/ld-version-script.txt"
+    LINKER_SCRIPT "${PROJECT_SOURCE_DIR}/source/ld-version-script.txt"
     SOURCES ${ARG_SOURCES}
     INCLUDE_DIRS
       ${ARG_INCLUDE_DIRS}
       ${sycl_inc_dir}
-    LIBRARIES ${ARG_LIBRARIES}
+    LIBRARIES
+      ${ARG_LIBRARIES}
+      OpenCL-Headers
   )
 
   install(TARGETS pi_${PLUGIN_NAME}

sycl/plugins/cuda/CMakeLists.txt

@@ -30,7 +30,6 @@ add_sycl_plugin(cuda
     "pi_cuda.hpp"
     "pi_cuda.cpp"
   LIBRARIES
-    OpenCL-Headers
     cudadrv
 )
 

sycl/plugins/esimd_emulator/CMakeLists.txt

@@ -109,7 +109,6 @@ add_sycl_plugin(esimd_emulator
     "pi_esimd_emulator.cpp"
   LIBRARIES
     "${LEVEL_ZERO_LOADER}"
-    OpenCL-Headers
     ${LIBCM}
     ${LIBIGFXCMRT_EMU}
     #TODO remove cyclic dependency

sycl/plugins/hip/CMakeLists.txt

@@ -18,8 +18,6 @@ add_sycl_plugin(hip
     "pi_hip.cpp"
   INCLUDE_DIRS
     ${sycl_plugin_dir}
-  LIBRARIES
-    OpenCL-Headers
 )
 set_target_properties(pi_hip PROPERTIES LINKER_LANGUAGE CXX)
 

sycl/plugins/level_zero/CMakeLists.txt

@@ -109,7 +109,6 @@ add_sycl_plugin(level_zero
     "${CMAKE_CURRENT_SOURCE_DIR}/usm_allocator.hpp"
   LIBRARIES
     "${LEVEL_ZERO_LOADER}"
-    OpenCL-Headers
     Threads::Threads
 )