Skip to content

[SYCL][CUDA][E2E] Disable tests failing on CUDA 13#22508

Merged
kswiecicki merged 3 commits into
intel:syclfrom
kekaczma:disable-cuda-13-failing-tests
Jul 14, 2026
Merged

[SYCL][CUDA][E2E] Disable tests failing on CUDA 13#22508
kswiecicki merged 3 commits into
intel:syclfrom
kekaczma:disable-cuda-13-failing-tests

Conversation

@kekaczma

@kekaczma kekaczma commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

After enabling CUDA 13 workflow (see #21564) few tests are failing sporadically:

  • WorkGroupMemory/basic_usage.cpp
  • bindless_images/read_sampled.cpp
  • bindless_images/vulkan_interop/vulkan_sycl_image_interop_read_1d.cpp
  • bindless_images/vulkan_interop/vulkan_sycl_image_interop_write_1d_unsampled.cpp

This PR implements CUDA version detection in LIT configuration and conditionally disables these 4 tests on CUDA 13+.

Tracked in:

@kekaczma kekaczma requested review from a team as code owners July 1, 2026 17:38
Tests disabled:
- WorkGroupMemory/basic_usage.cpp
- bindless_images/read_sampled.cpp
- bindless_images/vulkan_interop/vulkan_sycl_image_interop_read_1d.cpp
- bindless_images/vulkan_interop/vulkan_sycl_image_interop_write_1d_unsampled.cpp

Tracked in:
- intel#21806
- intel#21807
- intel#21808
@kekaczma kekaczma changed the title [SYCL][E2E] Disable tests failing on CUDA 13 [SYCL][CUDA][E2E] Disable tests failing on CUDA 13 Jul 1, 2026
@dyniols dyniols requested a review from Copilot July 3, 2026 08:23

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR temporarily disables a small set of SYCL end-to-end tests for the NVIDIA/CUDA target in response to sporadic failures observed after enabling the CUDA 13 CI workflow, with failures tracked in linked issues.

Changes:

  • Mark WorkGroupMemory/basic_usage.cpp as UNSUPPORTED on target-nvidia (tracked in #21806).
  • Mark bindless_images/read_sampled.cpp as UNSUPPORTED on target-nvidia (tracked in #21807).
  • Mark two Vulkan interop bindless image tests as UNSUPPORTED on target-nvidia (tracked in #21808).

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File Description
sycl/test-e2e/WorkGroupMemory/basic_usage.cpp Disables the test on NVIDIA/CUDA via UNSUPPORTED: target-nvidia with tracker #21806.
sycl/test-e2e/bindless_images/read_sampled.cpp Disables the test on NVIDIA/CUDA via UNSUPPORTED: target-nvidia with tracker #21807.
sycl/test-e2e/bindless_images/vulkan_interop/vulkan_sycl_image_interop_read_1d.cpp Disables the test on NVIDIA/CUDA via UNSUPPORTED: target-nvidia with tracker #21808.
sycl/test-e2e/bindless_images/vulkan_interop/vulkan_sycl_image_interop_write_1d_unsampled.cpp Disables the test on NVIDIA/CUDA via UNSUPPORTED: target-nvidia with tracker #21808.

Comment on lines +4 to +5
// UNSUPPORTED: target-nvidia
// UNSUPPORTED-TRACKER: https://github.com/intel/llvm/issues/21806
Comment on lines +6 to +7
// UNSUPPORTED: target-nvidia
// UNSUPPORTED-TRACKER: https://github.com/intel/llvm/issues/21807
Comment on lines +5 to +6
// UNSUPPORTED: target-nvidia
// UNSUPPORTED-TRACKER: https://github.com/intel/llvm/issues/21808
Comment on lines +9 to +10
// UNSUPPORTED: target-nvidia
// UNSUPPORTED-TRACKER: https://github.com/intel/llvm/issues/21808
@dyniols

dyniols commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

@kekaczma Do we have enabled CUDA 13 runner or not yet?
I checked and looks like PR #21564 changes has been reverted. We run on multiple CUDA versions and some of the pass making these tests unsupported means that they will be not run on any CUDA version.

Maybe adding some option for just disabling tests for specific CUDA version instead of just making test unsupported for all CUDA versions.

This commit implements CUDA version detection in LIT configuration and
conditionally disables 4 tests that are known to fail on CUDA 13+
@kekaczma

kekaczma commented Jul 3, 2026

Copy link
Copy Markdown
Contributor Author

@kekaczma Do we have enabled CUDA 13 runner or not yet? I checked and looks like PR #21564 changes has been reverted. We run on multiple CUDA versions and some of the pass making these tests unsupported means that they will be not run on any CUDA version.

Maybe adding some option for just disabling tests for specific CUDA version instead of just making test unsupported for all CUDA versions.

This PR is one of the last changes before enabling CUDA 13. I added CUDA version detection in the LIT configuration and conditionally disabled these 4 tests.

@dyniols

dyniols commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

@kekaczma Do we have enabled CUDA 13 runner or not yet? I checked and looks like PR #21564 changes has been reverted. We run on multiple CUDA versions and some of the pass making these tests unsupported means that they will be not run on any CUDA version.
Maybe adding some option for just disabling tests for specific CUDA version instead of just making test unsupported for all CUDA versions.

This PR is one of the last changes before enabling CUDA 13. I added CUDA version detection in the LIT configuration and conditionally disabled these 4 tests.

In that case I am okay with changes regarding bindless images.

@dyniols dyniols left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

bindless images changes LGTM.

@sys-ce-bb

Copy link
Copy Markdown
Contributor

@intel/llvm-gatekeepers please consider merging

@uditagarwal97

Copy link
Copy Markdown
Contributor

This commit will be authored by 162420515+kekaczma@users.noreply.github.com.

Hi @kekaczma,
Please check out https://llvm.org/docs/DeveloperPolicy.html#email-addresses - you'd have to change your Github account setting to allow committing from your personal/company email address. We don't allow committing from anonymous email addresses (like from noreply.github.com). Just changing your Github account settings should suffice - you don't have update the commits to include your personal email.
Thanks

@kswiecicki kswiecicki merged commit 5f96f3f into intel:sycl Jul 14, 2026
33 checks passed
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.

8 participants