Skip to content

[E2E] Enable end-to-end tests supported by the SPIR-V backend #20104

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 5 commits into
base: sycl
Choose a base branch
from
Open

[E2E] Enable end-to-end tests supported by the SPIR-V backend #20104

wants to merge 5 commits into from

Conversation

YixingZhang007
Copy link
Contributor

@YixingZhang007 YixingZhang007 commented Sep 16, 2025 •
edited
Loading

This PR re-enables SYCL end-to-end tests previously marked as XFAIL or UNSUPPORTED on the SPIR-V backend, as they now pass. For tests that are still failing, new issues have been created in llvm-project and added as XFAIL-TRACKER.

@YixingZhang007 YixingZhang007 requested review from a team as code owners September 16, 2025 18:03
@YixingZhang007 YixingZhang007 marked this pull request as draft September 16, 2025 18:03
@YixingZhang007 YixingZhang007 changed the title [SPIRV] Re-enable SYCL e2e tests marked XFAIL/Unsupported that pass with SPIR-V backend [E2E] Enable end-to-end tests supported by the SPIR-V backend Sep 16, 2025
aelovikov-intel
aelovikov-intel reviewed Sep 16, 2025
View reviewed changes
sycl/test-e2e/AddressSanitizer/aot/gpu.cpp Outdated
// RUN: %{run} not %t.out 2>&1 | FileCheck %S/Inputs/host-usm-oob.cpp

// XFAIL: spirv-backend && arch-intel_gpu_pvc
// XFAIL-TRACKER: CMPLRLLVM-64052
Copy link
Contributor

Choose a reason for hiding this comment

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

This is meaningless for the upstream.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for the suggestion! I've updated the xfail tracker to reference the issue in llvm-project instead of linking to the Jira issue.

@YixingZhang007 YixingZhang007 marked this pull request as ready for review September 26, 2025 17:42
@YixingZhang007 YixingZhang007 requested a review from a team as a code owner September 26, 2025 17:42
@steffenlarsen steffenlarsen removed the request for review from a team September 29, 2025 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aelovikov-intel aelovikov-intel aelovikov-intel approved these changes

@steffenlarsen steffenlarsen steffenlarsen approved these changes

@slawekptak slawekptak Awaiting requested review from slawekptak slawekptak is a code owner automatically assigned from intel/llvm-reviewers-runtime

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@YixingZhang007 @aelovikov-intel @steffenlarsen