Skip to content

Commit

Permalink
TSQR: Only tentatively enable CU{BLAS,SOLVER} if CUDA enabled
Browse files Browse the repository at this point in the history
Currently, there is no way to express that a TPL depends on another
TPL.  See discussion here:

trilinos#6496 (comment)

This means that we cannot use

TRIBITS_TPL_TENTATIVELY_ENABLE(CUBLAS)

(for example), unless we first protect it with IF(TPL_ENABLE_CUDA).
This commit does so for CUBLAS and CUSOLVER.
  • Loading branch information
Mark Hoemmen committed Jan 14, 2020
1 parent 3eadff1 commit 7996692
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions packages/tpetra/tsqr/cmake/Dependencies.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -7,5 +7,9 @@ SET(LIB_OPTIONAL_DEP_TPLS CUBLAS CUSOLVER)
SET(TEST_REQUIRED_DEP_TPLS)
SET(TEST_OPTIONAL_DEP_TPLS)

TRIBITS_TPL_TENTATIVELY_ENABLE(CUBLAS)
TRIBITS_TPL_TENTATIVELY_ENABLE(CUSOLVER)
IF(DEFINED TPL_ENABLE_CUDA)
IF(TPL_ENABLE_CUDA)
TRIBITS_TPL_TENTATIVELY_ENABLE(CUBLAS)
TRIBITS_TPL_TENTATIVELY_ENABLE(CUSOLVER)
ENDIF()
ENDIF()

0 comments on commit 7996692

Please sign in to comment.