Re-enable or delete dead code in test_external_semaphore.cpp

KhronosGroup / OpenCL-CTS

The OpenCL Conformance Tests

Apache License 2.0

184 stars 199 forks source link

Re-enable or delete dead code in test_external_semaphore.cpp #2109

Open shajder opened 1 month ago

shajder commented 1 month ago

While working on issue #1989 I've noticed in test_external_semaphore.cpp there is a function named test_external_semaphores_simple_2 which is not scheduled for test. Should this be removed or re-enabled?

joshqti commented 3 weeks ago

After reviewing and testing, I don't see any harm in enabling it. I have pushed a PR to enable it and delete some other dead code.

https://github.com/KhronosGroup/OpenCL-CTS/pull/2123