Skip to content

[SYCL] XFAIL ext_intel_cslice.cpp on DG2 #19811

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

Merged
merged 1 commit into from
Aug 18, 2025
Merged

Conversation

HPS-1
Copy link
Contributor

@HPS-1 HPS-1 commented Aug 15, 2025

This test is currently failing on Windows DG2 machines. The minimal reproducer is: minimal_reproducer.cpp
It turns out that removing "ZEX_NUMBER_OF_CCS=0:4" from the minimal reproducer makes it pass. This matches the comments in #18576 : the root cause here is that DG2 never supported ZEX_NUMBER_OF_CCS. Therefore, XFAIL-ing this test on all DG2 machines (previously it is only XFAIL-ed on Linux DG2 machines).

@HPS-1 HPS-1 temporarily deployed to WindowsCILock August 15, 2025 15:51 — with GitHub Actions Inactive
@HPS-1 HPS-1 temporarily deployed to WindowsCILock August 15, 2025 16:14 — with GitHub Actions Inactive
@HPS-1 HPS-1 temporarily deployed to WindowsCILock August 15, 2025 16:14 — with GitHub Actions Inactive
@HPS-1 HPS-1 marked this pull request as ready for review August 15, 2025 16:25
@HPS-1 HPS-1 requested a review from a team as a code owner August 15, 2025 16:25
@HPS-1
Copy link
Contributor Author

HPS-1 commented Aug 18, 2025

@intel/llvm-gatekeepers Hi can you please merge this? Thank you!

@aelovikov-intel aelovikov-intel merged commit e6de775 into intel:sycl Aug 18, 2025
28 checks passed
aelovikov-intel pushed a commit that referenced this pull request Aug 19, 2025
This test is currently failing on Windows DG2 machines. Same reasoning
as #19811: as mentioned in
#18576, the root cause here is that
DG2 never supported ZEX_NUMBER_OF_CCS. Therefore, XFAIL-ing this test on
all DG2 machines (previously it is only XFAIL-ed on Linux DG2 machines).

Signed-off-by: Hu, Peisen <[email protected]>
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.

3 participants