-
Notifications
You must be signed in to change notification settings - Fork 26
Update XFAILs of WaveActiveSum tests #527
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
bogner
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 other than the missing bug link
llvm-beanz
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.
One comment, feel free to take or leave it.
| # XFAIL: WARP && Clang | ||
|
|
||
| # Bug https://github.com/llvm/offload-test-suite/issues/525 | ||
| # XFAIL: NV && Clang & DirectX |
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.
Typo here. I've gone ahead and merged a fix: #536
| # XFAIL: NV && Clang & DirectX | |
| # XFAIL: NV && Clang && DirectX |
Changes the Clang XFAIL to a Vulkan && Clang XFAIL since the related issue (llvm/llvm-project#156775) appears to only be related to SPIR-V.
Also adds additional XFAILs for machines that still fail the tests.
Confirmed all Vulkan && Clang workflows XFAIL:
The remaining XFAILs are under D3D12 with Clang, mostly on NVIDIA and WARP. Most of the Metal XFAILs are expected except fp32 because fp32 passes with DXC.