From 33ffa18ccf43fe0f45d9802186f08cc3c9f358b2 Mon Sep 17 00:00:00 2001 From: viclafargue Date: Tue, 5 Mar 2024 15:01:31 +0100 Subject: [PATCH] REVERT cmake-format changes --- cpp/CMakeLists.txt | 61 +++++++++++++++++++++++------------------ cpp/test/CMakeLists.txt | 18 ++++++++++-- 2 files changed, 49 insertions(+), 30 deletions(-) diff --git a/cpp/CMakeLists.txt b/cpp/CMakeLists.txt index d0478c704..82213a3d8 100644 --- a/cpp/CMakeLists.txt +++ b/cpp/CMakeLists.txt @@ -239,13 +239,16 @@ target_include_directories( if(NOT BUILD_CPU_ONLY) # Keep cuVS as lightweight as possible. Only CUDA libs and rmm should be used in global target. - target_link_libraries( - cuvs - PUBLIC rmm::rmm $<$>:raft::raft> - $<$>:raft::compiled> - PRIVATE $<$:raft::raft> - $<$:raft::compiled_static> nvidia::cutlass::cutlass - ) + target_link_libraries(cuvs + PUBLIC + rmm::rmm + $<$>:raft::raft> + $<$>:raft::compiled> + PRIVATE + $<$:raft::raft> + $<$:raft::compiled_static> + nvidia::cutlass::cutlass + ) endif() # Endian detection @@ -300,14 +303,14 @@ endif() set_target_properties( cuvs - PROPERTIES BUILD_RPATH "\$ORIGIN" - INSTALL_RPATH "\$ORIGIN" - CXX_STANDARD 17 - CXX_STANDARD_REQUIRED ON - CUDA_STANDARD 17 - CUDA_STANDARD_REQUIRED ON + PROPERTIES BUILD_RPATH "\$ORIGIN" + INSTALL_RPATH "\$ORIGIN" + CXX_STANDARD 17 + CXX_STANDARD_REQUIRED ON + CUDA_STANDARD 17 + CUDA_STANDARD_REQUIRED ON INTERFACE_POSITION_INDEPENDENT_CODE ON - POSITION_INDEPENDENT_CODE ON + POSITION_INDEPENDENT_CODE ON ) target_compile_options( @@ -321,21 +324,24 @@ target_link_options(cuvs PRIVATE "${CMAKE_CURRENT_BINARY_DIR}/fatbin.ld") # * cuvs_c ------------------------------------------------------------------------------- if(BUILD_C_LIBRARY) add_library( - cuvs_c SHARED src/core/c_api.cpp src/neighbors/brute_force_c.cpp src/neighbors/ivf_flat_c.cpp - src/neighbors/ivf_pq_c.cpp src/neighbors/cagra_c.cpp + cuvs_c SHARED src/core/c_api.cpp + src/neighbors/brute_force_c.cpp + src/neighbors/ivf_flat_c.cpp + src/neighbors/ivf_pq_c.cpp + src/neighbors/cagra_c.cpp ) add_library(cuvs::c_api ALIAS cuvs_c) set_target_properties( cuvs_c - PROPERTIES BUILD_RPATH "\$ORIGIN" - INSTALL_RPATH "\$ORIGIN" - CXX_STANDARD 17 - CXX_STANDARD_REQUIRED ON - POSITION_INDEPENDENT_CODE ON + PROPERTIES BUILD_RPATH "\$ORIGIN" + INSTALL_RPATH "\$ORIGIN" + CXX_STANDARD 17 + CXX_STANDARD_REQUIRED ON + POSITION_INDEPENDENT_CODE ON INTERFACE_POSITION_INDEPENDENT_CODE ON - EXPORT_NAME c_api + EXPORT_NAME c_api ) target_compile_options(cuvs_c PRIVATE "$<$:${CUVS_CXX_FLAGS}>") @@ -346,11 +352,12 @@ if(BUILD_C_LIBRARY) INTERFACE "$" ) - target_link_libraries( - cuvs_c - PUBLIC cuvs::cuvs - PRIVATE $<$:raft::raft> - ) + target_link_libraries(cuvs_c + PUBLIC + cuvs::cuvs + PRIVATE + $<$:raft::raft> + ) # ensure CUDA symbols aren't relocated to the middle of the debug build binaries target_link_options(cuvs_c PRIVATE "${CMAKE_CURRENT_BINARY_DIR}/fatbin.ld") diff --git a/cpp/test/CMakeLists.txt b/cpp/test/CMakeLists.txt index 86186b919..841058f60 100644 --- a/cpp/test/CMakeLists.txt +++ b/cpp/test/CMakeLists.txt @@ -134,17 +134,29 @@ if(BUILD_C_TESTS) ConfigureTest(NAME INTEROP_TEST PATH test/core/interop.cu C_LIB) ConfigureTest( - NAME BRUTEFORCE_C_TEST PATH test/neighbors/run_brute_force_c.c test/neighbors/brute_force_c.cu + NAME + BRUTEFORCE_C_TEST + PATH + test/neighbors/run_brute_force_c.c + test/neighbors/brute_force_c.cu C_LIB ) ConfigureTest( - NAME IVF_FLAT_C_TEST PATH test/neighbors/run_ivf_flat_c.c test/neighbors/ann_ivf_flat_c.cu + NAME + IVF_FLAT_C_TEST + PATH + test/neighbors/run_ivf_flat_c.c + test/neighbors/ann_ivf_flat_c.cu C_LIB ) ConfigureTest( - NAME IVF_PQ_C_TEST PATH test/neighbors/run_ivf_pq_c.c test/neighbors/ann_ivf_pq_c.cu C_LIB + NAME + IVF_PQ_C_TEST + PATH + test/neighbors/run_ivf_pq_c.c + test/neighbors/ann_ivf_pq_c.cu C_LIB ) ConfigureTest(NAME CAGRA_C_TEST PATH test/neighbors/ann_cagra_c.cu C_LIB)