Skip to content

[SYCL][Graph] Fix post commit CI CUDA fail #19210

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

Open
wants to merge 1 commit into
base: sycl
Choose a base branch
from

Conversation

EwanC
Copy link
Contributor

@EwanC EwanC commented Jun 30, 2025

Graph/NativeCommand/cuda_record_async_malloc.cpp is failing in DPC++ post-commit CI https://github.com/intel/llvm/actions/runs/15963166914/job/45021127515

This is due to it running on CUDA 12.1, different CUDA version than pre-commit CI where cuStreamBeginCaptureToGraph is not available. This patch worksaround this issue by using the explicit graph building API to add the same sequence of nodes, which are available in CUDA version 12.1, see doc

`Graph/NativeCommand/cuda_record_async_malloc.cpp` is failing in DPC++
post-commit CI https://github.com/intel/llvm/actions/runs/15963166914/job/45021127515

This is due to it running on CUDA 12.1, different CUDA version than pre-commit CI
where `cuStreamBeginCaptureToGraph` is not available. This patch
worksaround this issue by using the explicit graph building API to add
the same sequence of nodes.
@EwanC EwanC marked this pull request as ready for review June 30, 2025 11:31
@EwanC EwanC requested a review from a team as a code owner June 30, 2025 11:31
@EwanC EwanC requested a review from reble June 30, 2025 11:31
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