[Piglit] [PATCH 8/9] cl: Add cospi tests

[Aaron Watry]

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',
>
>