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.

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

Add CUDA_STATIC_MATH_LIBRARIES #4526

Merged

Conversation

KyleFromNVIDIA
Copy link
Contributor

@KyleFromNVIDIA KyleFromNVIDIA commented Jul 8, 2024

Usage of the CUDA math libraries is independent of the CUDA runtime. Make their static/shared status separately controllable.

Contributes to rapidsai/build-planning#35

Usage of the CUDA math libraries is independent of the CUDA runtime.
Make their static/shared status separately controllable.
@KyleFromNVIDIA KyleFromNVIDIA added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change CMake labels Jul 8, 2024
@KyleFromNVIDIA KyleFromNVIDIA requested a review from a team as a code owner July 8, 2024 18:19
@KyleFromNVIDIA
Copy link
Contributor Author

/merge

@rapids-bot rapids-bot bot merged commit 2cabe02 into rapidsai:branch-24.08 Jul 9, 2024
131 checks passed
KyleFromNVIDIA added a commit to KyleFromNVIDIA/cugraph that referenced this pull request Aug 15, 2024
In rapidsai#4526, we introduced the
CUDA_STATIC_MATH_LIBRARIES option and set it to OFF by default. We
did not set it to ON in the Python builds, and as a result ended
up bundling dynamic CUDA libraries. Set the option to ON in the
Python build.
rapids-bot bot pushed a commit that referenced this pull request Aug 15, 2024
In #4526, we introduced the CUDA_STATIC_MATH_LIBRARIES option and set it to OFF by default. We did not set it to ON in the Python builds, and as a result ended up bundling dynamic CUDA libraries. Set the option to ON in the Python build.

Authors:
  - Kyle Edwards (https://github.com/KyleFromNVIDIA)

Approvers:
  - Vyas Ramasubramani (https://github.com/vyasr)

URL: #4612
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CMake cuGraph improvement Improvement / enhancement to an existing function non-breaking Non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants