-
Notifications
You must be signed in to change notification settings - Fork 131
Temporarily disabled unnormalized-clamp-linear-float.cpp #205
Conversation
@DenisBakhvalov - this test is already marked XFAIL for LevelZero and Windows. Why is it necessary to further disable it? If we don't want the test run at all, we can switch |
@cperkinsintel, based on what I saw, it sporadically passes on Windows + LevelZero, even though it is supposed to fail. |
@DenisBakhvalov , well let's just switch |
OK. |
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. Thanks!
Just ran checks in #202 one more time, and now this test fails with OpenCL on Linux: |
I think it's a different one. The failing test in your CI is |
@dm-vodopyanov - I'll take care of it. We had to adjust the nonnormalized test last week. I'll update this one too. The OCL:CPU has an error where it gets flaky results on certain edge cases. There's an outstanding ticket on it, but I'll adjust the test in the interim. |
Thanks a lot Chris! Denis, yes, sorry, this is completely another test, I didn't match the whole names. |
Related issue: intel/llvm#8814