fix the issue of symbols defined in omp not found (#53)

* fix the issue of symbols defined in omp not found Signed-off-by: jinjiabao.jjb <[email protected]>
antgroup · Oct 9, 2024 · eb1778a · eb1778a
1 parent 708e71c
commit eb1778a
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 6 deletions.
diff --git a/extern/mkl/mkl.cmake b/extern/mkl/mkl.cmake
@@ -59,8 +59,11 @@ if (CMAKE_HOST_SYSTEM_PROCESSOR STREQUAL "x86_64" AND ENABLE_INTEL_MKL)
     endforeach()
     message ("enable intel-mkl as blas backend")
 else ()
-    set (BLAS_LIBRARIES
-        libopenblas.a
-    )
+    set(BLAS_LIBRARIES libopenblas.a)
+    if (CMAKE_CXX_COMPILER_ID STREQUAL "Clang") 
+        list(PREPEND BLAS_LIBRARIES omp)
+    else() 
+        list(PREPEND BLAS_LIBRARIES gomp)
+    endif()
     message ("enable openblas as blas backend")
 endif ()
diff --git a/extern/roaringbitmap/roaringbitmap.cmake b/extern/roaringbitmap/roaringbitmap.cmake
@@ -22,12 +22,10 @@ if (DISABLE_AVX512_FORCE OR NOT COMPILER_AVX512_SUPPORTED)
   set (ROARING_DISABLE_AVX512 ON)
 endif ()
 
-set (CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-unused-function")
-set (CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -Wno-unused-function")
-
 # exclude roaringbitmap in vsag installation
 FetchContent_GetProperties(roaringbitmap)
 if(NOT roaringbitmap_POPULATED)
   FetchContent_Populate(roaringbitmap)
   add_subdirectory(${roaringbitmap_SOURCE_DIR} ${roaringbitmap_BINARY_DIR} EXCLUDE_FROM_ALL)
+  target_compile_options(roaring PRIVATE -Wno-unused-function)
 endif()