[Piglit] [PATCH 8/9] cl: Add cospi tests
Aaron Watry
awatry at gmail.com
Mon Sep 29 07:40:41 PDT 2014
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 <brunojimen at gmail.com> wrote:
> On Thu, 2014-09-25 at 15:01 -0500, Aaron Watry wrote:
>> Signed-off-by: Aaron Watry <awatry at gmail.com>
>> ---
>> 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',
>
>
More information about the Piglit
mailing list