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

Update FindSystemBLAS cmake #509

Merged
merged 3 commits into from
Apr 9, 2024
Merged

Update FindSystemBLAS cmake #509

Changes from 1 commit
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
38 changes: 26 additions & 12 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 Down Expand Up @@ -27,23 +26,42 @@ 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(BLAS_FOUND TRUE)
set(SystemBLAS_LIBRARIES USER_DEFINED)
else()
Rbiessy marked this conversation as resolved.
Show resolved Hide resolved
set(BLA_VENDOR OpenBLAS)
find_package(BLAS QUIET)
if(BLAS_FOUND)
set(OPENBLAS_FOUND TRUE)
set(SystemBLAS_LIBRARIES OPENBLAS_LIBRARIES)
Rbiessy marked this conversation as resolved.
Show resolved Hide resolved
endif()
Rbiessy marked this conversation as resolved.
Show resolved Hide resolved
if(NOT BLAS_FOUND)
message(WARNING "openBLAS library was not found on your system")
unset(BLA_VENDOR)
Rbiessy marked this conversation as resolved.
Show resolved Hide resolved
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_FOUND TRUE)
set(SystemBLAS_LIBRARIES BLAS_LIBRARIES)
endif()
endif()

if(NOT BLAS_FOUND)
set(BLA_STATIC ON)
find_package(BLAS QUIET)
set(SystemBLAS_FOUND TRUE)
set(SystemBLAS_LIBRARIES BLAS_LIBRARIES)
endif()

if(BLAS_FOUND AND NOT TARGET blas::blas)
Expand All @@ -53,10 +71,6 @@ 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
Expand Down
Loading