-
Notifications
You must be signed in to change notification settings - Fork 807
[SYCL][E2E] Enable USM/memops2d/* on Win #20249
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
This reverts commit 38d7194.
# https://github.com/llvm/llvm-project/issues/127791 | ||
config.unsupported_features += ['spirv-backend'] | ||
|
||
# https://github.com/intel/llvm/issues/8126 |
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.
Can we close #8126 as well, after this PR?
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'm not sure if these tests really don't fail anymore. Let's keep it for some time, I will close it a bit later if the issue actually fixed.
@intel/llvm-gatekeepers please merge |
These tests pass pre-commit & post-commit and I launched every test 100 times locally, though this testing might not be enough, I can't reproduce any failure. Let's try to enable them again and monitor the current status.