-
Notifications
You must be signed in to change notification settings - Fork 803
[SYCL][E2E] Add xfail to SG32/get_coordinate_ops.cpp test on WCL windows machine #20840
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
Conversation
| // SG size = 32 is not currently supported for SYCL Joint Matrix by IGC on DG2 | ||
| // UNSUPPORTED: gpu-intel-dg2 | ||
|
|
||
| // XFAIL: windows && (arch-intel_gpu_ptl_u || arch-intel_gpu_ptl_h || intel_gpu_lnl_m) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Both the Matrix/SG32/get_coordinate_ops.cpp and Matrix/element_wise_ops.cpp SYCL E2E tests appear to be passing when I run them on LNL Windows machine with driver version 20251112_160000 (Details can be found at Jira issue CMPLRLLVM-71724). I guess we might consider waiting for the confirmation from @wanyaman before making changes to these two tests.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sounds good.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I confirmed that the test still failed in 1206 Xmain nightly testing.
The driver version for reproducing this issue is IGFX: 101.8247.
|
|
|
Replaced by #21003 for Matrix/element_wise_ops.cpp |
YixingZhang007
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thank you :)
|
@intel/llvm-gatekeepers please consider merging |
|
@intel/llvm-gatekeepers, please consider merging this PR that has been approved and CI passed. |
|
could you please give the pr a more specific title? |
How about now? |
|
thanks! |
No description provided.