KhronosGroup / OpenCL-CTS

The OpenCL Conformance Tests
Apache License 2.0
178 stars 187 forks source link

math_brute_force: only test correctly rounded divide/sqrt for fp32 #1997

Open svenvh opened 6 days ago

svenvh commented 6 days ago

Skip the correctly rounded divide (divide_cr) and sqrt (sqrt_cr) tests for fp16 and fp64.

The corresponding build option to enable correctly rounded divide and sqrt is named -cl-fp32-correctly-rounded-divide-sqrt and the description refers only to "single precision floating-point", so this option should not apply to fp16 or fp64.

The specification states that fp16 and fp64 divide and sqrt must be correctly rounded for the full profile, without needing any additional build options. This is already tested by the regular divide and sqrt tests. For the embedded profile the ULP requirement is non-zero, but there is no build option to request a correctly rounded implementation anyway.

Fixes https://github.com/KhronosGroup/OpenCL-CTS/issues/1901 . Relates to https://github.com/KhronosGroup/OpenCL-CTS/issues/1996 .