Skip to content
This repository was archived by the owner on Mar 28, 2023. It is now read-only.

[SYCL] Temporary disable of DeviceLib tests #625

Merged
merged 3 commits into from
Dec 20, 2021

Conversation

denis-kabanov
Copy link

Disabled tests: SYCL/DeviceLib/built-ins/vector_relational.cpp, SYCL/DeviceLib/cmath_fp64_test.cpp, SYCL/DeviceLib/cmath_test.cpp, SYCL/DeviceLib/math_fp64_test.cpp, SYCL/DeviceLib/math_test.cpp

Disabled tests: SYCL/DeviceLib/built-ins/vector_relational.cpp, SYCL/DeviceLib/cmath_fp64_test.cpp, SYCL/DeviceLib/cmath_test.cpp, SYCL/DeviceLib/math_fp64_test.cpp, SYCL/DeviceLib/math_test.cpp
@vzakhari
Copy link

Can you please explain why you have to do it?

@denis-kabanov
Copy link
Author

Can you please explain why you have to do it?

All the last PRs to llvm-test-suite fail on these tests.

dm-vodopyanov
dm-vodopyanov previously approved these changes Dec 17, 2021
@dm-vodopyanov
Copy link

Can you also please disable Assert tests in separate PR? Please add links to these PRs to appropriate internal tickets.

@denis-kabanov
Copy link
Author

Can you also please disable Assert tests in separate PR? Please add links to these PRs to appropriate internal tickets.

I think disabling the assert tests is unnecessary because the latest PRs into llvm-test-suite don't fails on asserts anymore (maybe, they are already fixed).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants