ahmed/OpenCL-CTS
1
0
Fork 0
mirror of https://github.com/KhronosGroup/OpenCL-CTS.git synced 2026-03-19 06:09:01 +00:00
Code Issues Packages Projects Releases Wiki Activity

math_brute_force: fix exp/exp2 rlx ULP calculation (#1848)

Browse Source 
Fix the ULP error calculation for the `exp` and `exp2` builtins in
relaxed math mode for the full profile.

Previously, the `ulps` value kept being added to while verifying the
result buffer in a loop.  `ulps` could even become a `NaN` when the
input argument being tested was a `NaN`.

Signed-off-by: Sven van Haastregt <sven.vanhaastregt@arm.com>
This commit is contained in:
Sven van Haastregt
2024-01-09 18:50:34 +01:00
committed by GitHub
parent 6606fc2e7b
commit ebe8aa0fcc
1 changed files with 4 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
test_conformance/math_brute_force/unary_float.cpp
View File

@@ -303,15 +303,14 @@ cl_int Test(cl_uint job_id, cl_uint thread_id, void *data)
if (strcmp(fname, "exp") == 0 || strcmp(fname, "exp2") == 0)
{
float exp_error = ulps;
// For full profile, ULP depends on input value.
// For embedded profile, ULP comes from functionList.
if (!gIsEmbedded)
{
exp_error += floor(fabs(2 * s[j]));
ulps = 3.0f + floor(fabs(2 * s[j]));
}
fail = !(fabsf(err) <= exp_error);
ulps = exp_error;
fail = !(fabsf(err) <= ulps);
}
if (strcmp(fname, "tan") == 0)
{