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

Define PTDS for the stream hook libs #16182

Merged

Conversation

trxcllnt
Copy link
Contributor

@trxcllnt trxcllnt commented Jul 2, 2024

We must define CUDA_API_PER_THREAD_DEFAULT_STREAM for the stream hook lib, since cudaLaunchKernel in CUDA 12.4+ is now a macro that expands to a different function when it's not defined.

@trxcllnt trxcllnt added 3 - Ready for Review Ready for review by team improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Jul 2, 2024
@trxcllnt trxcllnt requested a review from a team as a code owner July 2, 2024 20:32
@github-actions github-actions bot added libcudf Affects libcudf (C++/CUDA) code. CMake CMake build issue labels Jul 2, 2024
@robertmaynard robertmaynard requested a review from vyasr July 3, 2024 13:49
@trxcllnt
Copy link
Contributor Author

trxcllnt commented Jul 8, 2024

/merge

@rapids-bot rapids-bot bot merged commit d9a3728 into rapidsai:branch-24.08 Jul 8, 2024
85 checks passed
@trxcllnt trxcllnt deleted the fix/define-ptds-for-stream-libs branch July 8, 2024 16:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3 - Ready for Review Ready for review by team CMake CMake build issue improvement Improvement / enhancement to an existing function libcudf Affects libcudf (C++/CUDA) code. non-breaking Non-breaking change
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants