Remove UMF from ur::common

and make a separate ur_umf interface library.
This way, components that do not require UMF (e.g. loader)
do not need to link with it.

Author: igchor

source/adapters/cuda/CMakeLists.txt

@@ -114,6 +114,7 @@ endif()
 target_link_libraries(${TARGET_NAME} PRIVATE
     ${PROJECT_NAME}::headers
     ${PROJECT_NAME}::common
+    ${PROJECT_NAME}::umf
     Threads::Threads
     cudadrv
     ${EXTRA_LIBS}

source/adapters/hip/CMakeLists.txt

@@ -140,6 +140,7 @@ if("${UR_HIP_PLATFORM}" STREQUAL "AMD")
     target_link_libraries(${TARGET_NAME} PRIVATE
     ${PROJECT_NAME}::headers
     ${PROJECT_NAME}::common
+    ${PROJECT_NAME}::umf
     rocmdrv
     )
 
@@ -174,6 +175,7 @@ elseif("${UR_HIP_PLATFORM}" STREQUAL "NVIDIA")
     target_link_libraries(${TARGET_NAME} PRIVATE
     ${PROJECT_NAME}::headers
     ${PROJECT_NAME}::common
+    ${PROJECT_NAME}::umf
     Threads::Threads
     cudadrv
     cudart

source/adapters/level_zero/CMakeLists.txt

@@ -166,6 +166,7 @@ endif()
 target_link_libraries(${TARGET_NAME} PRIVATE
     ${PROJECT_NAME}::headers
     ${PROJECT_NAME}::common
+    ${PROJECT_NAME}::umf
     LevelZeroLoader
     LevelZeroLoader-Headers
 )

source/adapters/native_cpu/CMakeLists.txt

@@ -52,6 +52,7 @@ find_package(Threads REQUIRED)
 target_link_libraries(${TARGET_NAME} PRIVATE
         ${PROJECT_NAME}::headers
         ${PROJECT_NAME}::common
+        ${PROJECT_NAME}::umf
         Threads::Threads
 )
 

source/adapters/opencl/CMakeLists.txt

@@ -96,6 +96,7 @@ target_include_directories(${TARGET_NAME} PRIVATE
 target_link_libraries(${TARGET_NAME} PRIVATE
     ${PROJECT_NAME}::headers
     ${PROJECT_NAME}::common
+    ${PROJECT_NAME}::umf
     Threads::Threads
     ${OpenCLICDLoaderLibrary}
 )

source/common/CMakeLists.txt

@@ -4,9 +4,6 @@
 # SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
 
 add_ur_library(ur_common STATIC
-    umf_helpers.hpp
-    umf_pools/disjoint_pool_config_parser.cpp
-    ur_pool_manager.hpp
     ur_util.cpp
     ur_util.hpp
     $<$<PLATFORM_ID:Windows>:windows/ur_lib_loader.cpp>
@@ -52,9 +49,6 @@ FetchContent_MakeAvailable(unified-memory-framework)
 FetchContent_GetProperties(unified-memory-framework)
 
 target_link_libraries(ur_common PUBLIC
-    umf::umf
-    umf::headers
-    umf::disjoint_pool
     ${CMAKE_DL_LIBS}
     ${PROJECT_NAME}::headers
 )
@@ -72,3 +66,18 @@ install(TARGETS ur_common
     RUNTIME DESTINATION ${CMAKE_INSTALL_BINDIR}
     LIBRARY DESTINATION ${CMAKE_INSTALL_LIBDIR}
 )
+
+add_library(ur_umf INTERFACE)
+target_sources(ur_umf INTERFACE
+    umf_helpers.hpp
+    umf_pools/disjoint_pool_config_parser.cpp
+    ur_pool_manager.hpp
+)
+
+add_library(${PROJECT_NAME}::umf ALIAS ur_umf)
+
+target_link_libraries(ur_umf INTERFACE
+    umf::umf
+    umf::headers
+    umf::disjoint_pool
+)

test/adapters/cuda/CMakeLists.txt

@@ -28,4 +28,4 @@ target_include_directories(test-adapter-cuda PRIVATE
     ${PROJECT_SOURCE_DIR}/source/adapters/cuda
 )
 
-target_link_libraries(test-adapter-cuda PRIVATE cudadrv)
+target_link_libraries(test-adapter-cuda PRIVATE cudadrv ${PROJECT_NAME}::umf)

test/adapters/hip/CMakeLists.txt

@@ -26,4 +26,4 @@ target_compile_definitions(test-adapter-hip PRIVATE
     ${HIP_COMPILE_DEFINITIONS}
 )
 
-target_link_libraries(test-adapter-hip PRIVATE rocmdrv)
+target_link_libraries(test-adapter-hip PRIVATE rocmdrv ${PROJECT_NAME}::umf)

test/adapters/level_zero/v2/CMakeLists.txt

@@ -23,6 +23,7 @@ function(add_unittest name)
 
     target_link_libraries(${target} PRIVATE
         ${PROJECT_NAME}::common
+        ${PROJECT_NAME}::umf
         LevelZeroLoader
         LevelZeroLoader-Headers
     )

test/usm/CMakeLists.txt

@@ -15,6 +15,7 @@ function(add_usm_test name)
         PRIVATE
         ${PROJECT_NAME}::common
         ${PROJECT_NAME}::loader
+        ${PROJECT_NAME}::umf
         ur_testing
         GTest::gtest_main)
     add_test(NAME usm-${name}