-
Notifications
You must be signed in to change notification settings - Fork 734
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
[SYCL] Several tests failed with OpenCL FPGA in precommit #12683
Comments
As I understand Regarding Assert/* tests - it looks like FileCheck should check
See the output:
@aelovikov-intel are we ignoring this - BTW. this link |
I think it should be |
This PR enables testing on opencl::fpga for Linux in postcommit. I have currently disabled the tests that were failing on the fpga and created another Github issue for the same: #12683. I am not very sure if the failures are because of some implementation bug or if that feature is not supported on fpga.
…cpp` Both `accelerator` and `cpu` are unsupported due to the same reason, there is no unique FPGA-related bug here. Closes intel#12683.
Describe the bug
Testing in #12673 shows the following tests failed with opencl:fpga. Assert tests failed because the tests did not produce the desired output.
Affected tests:
Assert/assert_in_kernels.cppAssert/assert_in_multiple_tus.cppAssert/assert_in_multiple_tus_one_ndebug.cppAssert/assert_in_one_kernel.cppAssert/assert_in_simultaneous_kernels.cppAssert/assert_in_simultaneously_multiple_tus.cppDeviceLib/string_test.cpp
Matrix/joint_matrix_opt_kernel_feature_unsupported_hw.cppsyclcompat/atomic/atomic_arith.cppsyclcompat/atomic/atomic_comp_exchange.cppDeviceLib/string_test failed with the following error:
Detailed test logs: here
To Reproduce
Run sycl-e2e tests on a machine with FPGA enabled. No special LIT/CMake option needed to reproduce these failures.
*Environment
Update:- The testing infrastructure of Assert/* tests and joint_matrix test was updated and these tests are no longer failing on FPGA.
Update#2:- syclcompact/Atomic/* tests were fixed. See #12673 (review)
The text was updated successfully, but these errors were encountered: