Skip to content

Fix CUDA 13 API issues.#314

Merged
LegNeato merged 1 commit intoRust-GPU:mainfrom
nnethercote:cuda-13-api
Nov 24, 2025
Merged

Fix CUDA 13 API issues.#314
LegNeato merged 1 commit intoRust-GPU:mainfrom
nnethercote:cuda-13-api

Conversation

@nnethercote
Copy link
Copy Markdown
Collaborator

Four functions we use changed type signature in CUDA 13.0. This commit adds a cfg for each one and then uses conditional compilation at each call site so that the call works for both 12.x and 13.0.

Four functions we use changed type signature in CUDA 13.0. This commit
adds a cfg for each one and then uses conditional compilation at each
call site so that the call works for both 12.x and 13.0.
@nnethercote nnethercote mentioned this pull request Nov 21, 2025
Copy link
Copy Markdown
Contributor

@LegNeato LegNeato left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we get 13 added to CI to make sure this works? It should be a matter of copying the 12 ones, changing the base image, adding to the yaml. If you'd prefer to not include that in this diff, that's fine.

@nnethercote
Copy link
Copy Markdown
Collaborator Author

Can we get 13 added to CI to make sure this works?

CUDA 13 doesn't work yet. There is one more change I need to make so that the gemm example works. After I've done that we can enable it in CI. I've added an item to my todo list.

So I think this can be merged as is.

@LegNeato LegNeato merged commit 2623e21 into Rust-GPU:main Nov 24, 2025
17 of 18 checks passed
@nnethercote nnethercote deleted the cuda-13-api branch November 24, 2025 20:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants