Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update FindSystemBLAS cmake #509

Merged
merged 3 commits into from
Apr 9, 2024
Merged

Update FindSystemBLAS cmake #509

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
2eece9f
Update FindSystemBLAS cmake
s-Nick Apr 3, 2024
268ff13
Update message in cmake when a BLAS Library is set & minor fixes.
s-Nick Apr 8, 2024
08d78b4
Completely remove SystemBLAS_FOUND variable and FOUND_VAR
s-Nick Apr 8, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
37 changes: 22 additions & 15 deletions cmake/Modules/FindSystemBLAS.cmake
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@

#/***************************************************************************
# *
# * @license
Expand All @@ -23,27 +22,40 @@
# * @filename FindSystemBLAS.cmake
# *
# **************************************************************************/
set(SystemBLAS_FOUND FALSE)

include(FindPackageHandleStandardArgs)

find_library(OPENBLAS_LIBRARIES NAMES openblas libopenblas)
find_path(OPENBLAS_INCLUDE_DIRS openblas_config.h)
if(OPENBLAS_LIBRARIES AND OPENBLAS_INCLUDE_DIRS)
# Let user possibility to set blas path to similar libraries like lapack
if(BLAS_LIBRARIES AND BLAS_INCLUDE_DIRS)
find_package(Threads REQUIRED)
add_library(blas::blas UNKNOWN IMPORTED)
set_target_properties(blas::blas PROPERTIES
INTERFACE_INCLUDE_DIRECTORIES "${OPENBLAS_INCLUDE_DIRS}"
INTERFACE_INCLUDE_DIRECTORIES "${BLAS_INCLUDE_DIRS}"
INTERFACE_LINK_LIBRARIES Threads::Threads
IMPORTED_LOCATION "${OPENBLAS_LIBRARIES}"
IMPORTED_LOCATION "${BLAS_LIBRARIES}"
)
set(OPENBLAS_FOUND TRUE)
set(SystemBLAS_LIBRARIES OPENBLAS_LIBRARIES)
set(SystemBLAS_LIBRARIES ${BLAS_LIBRARIES})
else()
set(BLA_VENDOR OpenBLAS)
find_package(BLAS QUIET)
if(BLAS_FOUND)
set(SystemBLAS_LIBRARIES ${BLAS_LIBRARIES})
endif()
if(NOT BLAS_FOUND)
message(WARNING "openBLAS library was not found on your system")
unset(BLA_VENDOR)
set(BLA_VENDOR All)
find_package(BLAS QUIET)
if (BLAS_FOUND)
message(WARNING "Found another BLAS library on your system. Not using openBLAS may cause some tests to fail.")
message("-- BLAS library found at ${BLAS_LIBRARIES}")
set(SystemBLAS_LIBRARIES ${BLAS_LIBRARIES})
endif()
endif()

if(NOT BLAS_FOUND)
set(BLA_STATIC ON)
find_package(BLAS QUIET)
set(SystemBLAS_LIBRARIES ${BLAS_LIBRARIES})
endif()

if(BLAS_FOUND AND NOT TARGET blas::blas)
Expand All @@ -53,12 +65,7 @@ else()
)
endif()

if(BLAS_FOUND)
set(SystemBLAS_FOUND TRUE)
set(SystemBLAS_LIBRARIES BLAS_LIBRARIES)
Rbiessy marked this conversation as resolved.
Show resolved Hide resolved
endif()
endif()

find_package_handle_standard_args(SystemBLAS
FOUND_VAR SystemBLAS_FOUND
REQUIRED_VARS SystemBLAS_LIBRARIES)
Loading