Avoid recompiling DPCTLSyclInterface sources for coverage tests

llvm-cov is able to generate coverage for code in shared object library
so long of object files used to link it are passed with -object argument
to `llvm-cov report`.

These arguments are generated using CMake generator expression, specifically
TARGET_OBJECTS:DPCTLSyclInterface.

Author: oleksandr-pavlyk

libsyclinterface/CMakeLists.txt

@@ -156,12 +156,42 @@ file(GLOB_RECURSE helper_sources
     ${CMAKE_CURRENT_SOURCE_DIR}/helper/source/*.cpp
 )
 
+# Enable code coverage related settings
+if(DPCTL_GENERATE_COVERAGE)
+    # check if lcov is available
+    find_package(Lcov REQUIRED)
+    # check if llvm-cov version 11 is available
+    find_package(LLVMCov 11 REQUIRED)
+    # check if llvm-profdata is available
+    find_package(LLVMProfdata REQUIRED)
+    # Turn on DPCTL_BUILD_CAPI_TESTS as building tests is needed to generate
+    # coverage reports.
+    set(DPCTL_BUILD_CAPI_TESTS "ON")
+    if(DPCTL_COVERAGE_REPORT_OUTPUT_DIR)
+        set(COVERAGE_OUTPUT_DIR ${DPCTL_COVERAGE_REPORT_OUTPUT_DIR})
+    else()
+        set(COVERAGE_OUTPUT_DIR ${CMAKE_CURRENT_BINARY_DIR})
+    endif()
+    message(STATUS "Coverage reports to be saved at ${COVERAGE_OUTPUT_DIR}")
+
+    # Add profiling flags
+    set(CMAKE_CXX_FLAGS
+        "${CMAKE_CXX_FLAGS} -fprofile-instr-generate -fcoverage-mapping -fno-sycl-use-footer -DDPCTL_COVERAGE"
+    )
+endif()
+
 add_library(DPCTLSyclInterface
     SHARED
     ${sources}
     ${helper_sources}
 )
 
+if(DPCTL_GENERATE_COVERAGE)
+    target_link_options(DPCTLSyclInterface
+        PRIVATE -fprofile-instr-generate -fcoverage-mapping
+    )
+endif()
+
 target_include_directories(DPCTLSyclInterface
     PUBLIC
     ${CMAKE_CURRENT_SOURCE_DIR}/include/
@@ -247,25 +277,6 @@ install(
     DESTINATION ${_include_destination}/Config
 )
 
-# Enable code coverage related settings
-if(DPCTL_GENERATE_COVERAGE)
-    # check if lcov is available
-    find_package(Lcov REQUIRED)
-    # check if llvm-cov version 11 is available
-    find_package(LLVMCov 11 REQUIRED)
-    # check if llvm-profdata is available
-    find_package(LLVMProfdata REQUIRED)
-    # Turn on DPCTL_BUILD_CAPI_TESTS as building tests is needed to generate
-    # coverage reports.
-    set(DPCTL_BUILD_CAPI_TESTS "ON")
-    if(DPCTL_COVERAGE_REPORT_OUTPUT_DIR)
-        set(COVERAGE_OUTPUT_DIR ${DPCTL_COVERAGE_REPORT_OUTPUT_DIR})
-    else()
-        set(COVERAGE_OUTPUT_DIR ${CMAKE_CURRENT_BINARY_DIR})
-    endif()
-    message(STATUS "Coverage reports to be saved at ${COVERAGE_OUTPUT_DIR}")
-endif()
-
 # Add sub-directory to build the dpctl C API test cases
 if(DPCTL_BUILD_CAPI_TESTS)
     add_subdirectory(tests)

libsyclinterface/tests/CMakeLists.txt

@@ -25,30 +25,12 @@ if(DPCTL_GENERATE_COVERAGE)
     file(GLOB_RECURSE
         sources ${CMAKE_CURRENT_SOURCE_DIR}/*.cpp
     )
-    file(GLOB_RECURSE
-        dpctl_sources ${CMAKE_CURRENT_SOURCE_DIR}/../source/*.cpp
-    )
-    file(GLOB_RECURSE
-        helper_sources ${CMAKE_CURRENT_SOURCE_DIR}/../helper/source/*.cpp
-    )
-    # Exclude from sources
-    list(REMOVE_ITEM
-        dpctl_sources
-        "${CMAKE_CURRENT_SOURCE_DIR}/../source/dpctl_vector_templ.cpp"
-    )
-
-    # Add profiling flags
-    set(CMAKE_CXX_FLAGS
-        "${CMAKE_CXX_FLAGS} -fprofile-instr-generate -fcoverage-mapping -fno-sycl-use-footer -DDPCTL_COVERAGE"
-    )
 
     # Add all dpctl sources into a single executable so that we can run coverage
     # analysis and generate a report.
     add_executable(dpctl_c_api_tests
         EXCLUDE_FROM_ALL
         ${sources}
-        ${dpctl_sources}
-        ${helper_sources}
     )
     target_include_directories(dpctl_c_api_tests
         PRIVATE "${CMAKE_CURRENT_SOURCE_DIR}/../helper/include"
@@ -57,9 +39,11 @@ if(DPCTL_GENERATE_COVERAGE)
     target_link_libraries(dpctl_c_api_tests
         ${CMAKE_THREAD_LIBS_INIT}
         GTest::GTest
+        DPCTLSyclInterface
         ${IntelSycl_OPENCL_LIBRARY}
         ${CMAKE_DL_LIBS}
     )
+    set(object_arg "-object;")
     add_custom_target(llvm-cov
         COMMAND ${CMAKE_COMMAND} -E env DPCTL_VERBOSITY=warning ${CMAKE_CURRENT_BINARY_DIR}/dpctl_c_api_tests
         COMMAND ${LLVMProfdata_EXE}
@@ -71,10 +55,10 @@ if(DPCTL_GENERATE_COVERAGE)
             report
             ${CMAKE_CURRENT_BINARY_DIR}/dpctl_c_api_tests
             -instr-profile=dpctl.profdata
-            ${dpctl_sources}
-            ${helper_sources}
+            "${object_arg}$<JOIN:$<TARGET_OBJECTS:DPCTLSyclInterface>,;${object_arg}>"
         WORKING_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}
-	DEPENDS dpctl_c_api_tests
+        COMMAND_EXPAND_LISTS
+        DEPENDS dpctl_c_api_tests
     )
 
     add_custom_target(lcov-genhtml
@@ -90,7 +74,7 @@ if(DPCTL_GENERATE_COVERAGE)
             --output-directory
             ${COVERAGE_OUTPUT_DIR}/dpctl-c-api-coverage
         WORKING_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}
-	DEPENDS llvm-cov
+        DEPENDS llvm-cov
     )
 else()
     file(GLOB_RECURSE sources ${CMAKE_CURRENT_SOURCE_DIR}/*.cpp)