Re: [Piglit] [PATCH 03/10] cl: Add tests for different versions of fmin / fmax.

> On Dec 5, 2016, at 13:35, Jan Vesely wrote: > > On Mon, 2016-12-05 at 09:48 -0800, arse...@gmail.com > wrote: >> From: Matt Arsenault >> >> These do not use the normal simple format because the number >> of combinations that need to be tested is simply too large,

Re: [Piglit] [PATCH] cl: Add test for clz optimizations

On Tue, 2016-12-06 at 11:15 -0800, arse...@gmail.com wrote: > From: Matt Arsenault > > v2: Rename test Reviewed-by: Jan Vesely and pushed (with slight rename followup) Jan > --- > tests/cl/program/execute/clz-optimizations.cl | 389 > ++ > 1 file changed, 389 inserti

Re: [Piglit] [PATCH] cl: Add test for f64 inline immediates

On Tue, 2016-12-06 at 11:14 -0800, arse...@gmail.com wrote: > From: Matt Arsenault > > v2: Require cl_khr_fp64 Reviewed-by: Jan Vesely and pushed. Jan > --- > .../execute/amdgcn-f64-inline-immediates.cl| 270 > + > 1 file changed, 270 insertions(+) > create mode

Re: [Piglit] [PATCH] cl: Add f32 immediate tests

On Tue, 2016-12-06 at 11:14 -0800, arse...@gmail.com wrote: > From: Matt Arsenault > > --- > .../execute/amdgcn-f32-inline-immediates.cl| 237 > + > 1 file changed, 237 insertions(+) > create mode 100644 tests/cl/program/execute/amdgcn-f32-inline-immediates.cl > >

Re: [Piglit] [PATCH 02/10] cl: Add sign_extend_inreg test

On Tue, 2016-12-06 at 13:33 -0800, Matt Arsenault wrote: > > On Dec 6, 2016, at 12:18, Jan Vesely wrote: > > > > On Tue, 2016-12-06 at 10:55 -0800, Matt Arsenault wrote: > > > > On Dec 5, 2016, at 13:20, Jan Vesely > > > > wrote: > > > > > > > > On Mon, 2016-12-05

Re: [Piglit] [PATCH 02/10] cl: Add sign_extend_inreg test

> On Dec 6, 2016, at 12:18, Jan Vesely wrote: > > On Tue, 2016-12-06 at 10:55 -0800, Matt Arsenault wrote: >>> On Dec 5, 2016, at 13:20, Jan Vesely >> > wrote: >>> >>> On Mon, 2016-12-05 at 09:48 -0800, arse...@gmail.com >>>

Re: [Piglit] [PATCH 02/10] cl: Add sign_extend_inreg test

On Tue, 2016-12-06 at 10:55 -0800, Matt Arsenault wrote: > > On Dec 5, 2016, at 13:20, Jan Vesely wrote: > > > > On Mon, 2016-12-05 at 09:48 -0800, arse...@gmail.com > > wrote: > > > From: Matt Arsenault mailto:arse...@gmail.com>> > > > > > > --- > > > tests/cl/progra

Re: [Piglit] [PATCH 03/10] cl: Add tests for different versions of fmin / fmax.

On Tue, 2016-12-06 at 11:12 -0800, Matt Arsenault wrote: > > On Dec 5, 2016, at 13:35, Jan Vesely wrote: > > > > On Mon, 2016-12-05 at 09:48 -0800, arse...@gmail.com > > wrote: > > > From: Matt Arsenault > > > > > > These do not use the normal simple format because t

[Piglit] [PATCH] cl: Add test for clz optimizations

From: Matt Arsenault v2: Rename test --- tests/cl/program/execute/clz-optimizations.cl | 389 ++ 1 file changed, 389 insertions(+) create mode 100644 tests/cl/program/execute/clz-optimizations.cl diff --git a/tests/cl/program/execute/clz-optimizations.cl b/tests/cl/pro

[Piglit] [PATCH] cl: Add f32 immediate tests

From: Matt Arsenault --- .../execute/amdgcn-f32-inline-immediates.cl| 237 + 1 file changed, 237 insertions(+) create mode 100644 tests/cl/program/execute/amdgcn-f32-inline-immediates.cl diff --git a/tests/cl/program/execute/amdgcn-f32-inline-immediates.cl b/tests/

[Piglit] [PATCH] cl: Add test for f64 inline immediates

From: Matt Arsenault v2: Require cl_khr_fp64 --- .../execute/amdgcn-f64-inline-immediates.cl| 270 + 1 file changed, 270 insertions(+) create mode 100644 tests/cl/program/execute/amdgcn-f64-inline-immediates.cl diff --git a/tests/cl/program/execute/amdgcn-f64-inline

Re: [Piglit] [PATCH 07/10] cl: Add test for negative index + small offset for private

> On Dec 6, 2016, at 11:04, Jan Vesely wrote: > > On Tue, 2016-12-06 at 10:52 -0800, Matt Arsenault wrote: >>> On Dec 5, 2016, at 12:42, Jan Vesely wrote: >>> >>> On Mon, 2016-12-05 at 09:48 -0800, arse...@gmail.com >>> wrote: From: Matt Arsenault -

Re: [Piglit] [PATCH 07/10] cl: Add test for negative index + small offset for private

On Tue, 2016-12-06 at 10:52 -0800, Matt Arsenault wrote: > > On Dec 5, 2016, at 12:42, Jan Vesely wrote: > > > > On Mon, 2016-12-05 at 09:48 -0800, arse...@gmail.com > > wrote: > > > From: Matt Arsenault > > > > > > --- > > > .../execute/negative-private-base-pointer

Re: [Piglit] [PATCH 09/10] Require python 2.7

> On Dec 5, 2016, at 12:30, Jan Vesely wrote: > > On Mon, 2016-12-05 at 09:48 -0800, arse...@gmail.com wrote: >> From: Matt Arsenault >> >> Things seem to break with python3 > > can you be more specific? things have run OK for me with python3.5 for > some time. > > Jan I”ve had this patch s

Re: [Piglit] [PATCH 01/10] cl: Add tests for 24-bit div / rem optimization

> On Dec 5, 2016, at 13:24, Jan Vesely wrote: > > On Mon, 2016-12-05 at 09:48 -0800, arse...@gmail.com wrote: >> From: Matt Arsenault >> >> --- >> tests/cl/program/execute/scalar-arithmetic-int.cl | 1657 >> + >> 1 file changed, 1657 insertions(+) > > I have used an extend

Re: [Piglit] [PATCH 06/10] cl: Add test for clz

> On Dec 5, 2016, at 12:14, Jan Vesely wrote: > > On Mon, 2016-12-05 at 09:48 -0800, arse...@gmail.com > wrote: >> From: Matt Arsenault >> >> --- >> tests/cl/program/execute/clz.cl | 389 >> >> 1 file changed, 389 insertions(

Re: [Piglit] [PATCH 02/10] cl: Add sign_extend_inreg test

> On Dec 5, 2016, at 13:20, Jan Vesely wrote: > > On Mon, 2016-12-05 at 09:48 -0800, arse...@gmail.com > wrote: >> From: Matt Arsenault mailto:arse...@gmail.com>> >> >> --- >> tests/cl/program/execute/sign_extend_inreg.cl | 387 >> ++ >> 1 fil

Re: [Piglit] [PATCH 07/10] cl: Add test for negative index + small offset for private

> On Dec 5, 2016, at 12:42, Jan Vesely wrote: > > On Mon, 2016-12-05 at 09:48 -0800, arse...@gmail.com > wrote: >> From: Matt Arsenault >> >> --- >> .../execute/negative-private-base-pointer.cl | 120 >> + >> 1 file changed, 120 insertions(

[Piglit] [PATCH] GL_ARB_draw_instanced/dlist: Fix crash

From: Arthur Huillet The test was passing a NULL indices pointer without an element buffer bound. Nothing in the GL spec requires a NULL check, so drivers were free to crash before checking the error condition that the test attempts to check. Fix this by passing an index buffer. Also swap ver