> On Dec 5, 2016, at 12:14, Jan Vesely <[email protected]> wrote:
>
> On Mon, 2016-12-05 at 09:48 -0800, [email protected]
> <mailto:[email protected]> wrote:
>> From: Matt Arsenault <[email protected]>
>>
>> ---
>> tests/cl/program/execute/clz.cl | 389
>> ++++++++++++++++++++++++++++++++++++++++
>> 1 file changed, 389 insertions(+)
>> create mode 100644 tests/cl/program/execute/clz.cl
>>
>> diff --git a/tests/cl/program/execute/clz.cl
>> b/tests/cl/program/execute/clz.cl
>> new file mode 100644
>> index 0000000..06ba0e3
>> --- /dev/null
>> +++ b/tests/cl/program/execute/clz.cl
>> @@ -0,0 +1,389 @@
>> +/*!
>> +
>> +[config]
>> +name: clz
>> +clc_version_min: 10
>
> there already is a generated clz test, what's wrong with adding
> testcases there?
>
> Jan
I looked at this briefly and it looks like it only is testing the builtin
function over various ranges. This is more targeted for optimizations involving
clz rather than just the raw function itself, so maybe it should be renamed. It
test combines like folding compare + select with 0 input argument into the
behavior of v_ffbh_u32. I’m not sure this would fit in easily with the simple
function tests
-Matt
_______________________________________________
Piglit mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/piglit
