Yup, looks like I messed that up. I Could've sworn that I had run the generated tests for all of the built-ins in this series, but that must not be the case.
I'll send a v2 of the last 2 tests. --Aaron On Sun, Sep 28, 2014 at 11:46 AM, Bruno Jimenez <[email protected]> wrote: > On Thu, 2014-09-25 at 15:01 -0500, Aaron Watry wrote: >> Signed-off-by: Aaron Watry <[email protected]> >> --- >> generated_tests/generate-cl-math-builtins.py | 10 ++++++++++ >> 1 file changed, 10 insertions(+) >> >> diff --git a/generated_tests/generate-cl-math-builtins.py >> b/generated_tests/generate-cl-math-builtins.py >> index 531106d..9f5d75c 100644 >> --- a/generated_tests/generate-cl-math-builtins.py >> +++ b/generated_tests/generate-cl-math-builtins.py >> @@ -42,6 +42,7 @@ CLC_VERSION_MIN = { >> 'copysign' : 10, >> 'cos' : 10, >> 'cosh' : 10, >> + 'cospi' : 10, >> 'fabs' : 10, >> 'floor' : 10, >> 'fmod' : 10, >> @@ -172,6 +173,15 @@ tests = { >> ], >> 'tolerance' : 4 >> }, >> + 'cospi' : { >> + 'arg_types' : [F, F], >> + 'function_type': 'ttt', >> + 'values' : [ >> + [1.0, cos(pi*pi/2), cos(pi*3*pi/2), cos(2*pi*pi), >> cos(pi*1.12345)], # Result >> + [0.0, pi / 2, pi, 3 * pi / 2, 2 * pi, 1.12345 >> ] # Arg0 > ^^^^ > Hi, > > I think that you have a stray 'pi' here. (And in the following patch > too) > > -Bruno > >> + ], >> + 'tolerance' : 4 >> + }, >> 'fabs' : { >> 'arg_types' : [F, F], >> 'function_type': 'ttt', > > _______________________________________________ Piglit mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/piglit
