Re: [Piglit] [PATCH 2/2] cl: Change data types of char/short buffers in integer limits tests

2015-09-25 Thread Jan Vesely
On Tue, 2015-09-22 at 09:11 -0500, Aaron Watry wrote: > On Tue, Sep 22, 2015 at 12:02 AM, Jan Vesely > wrote: > > > On Mon, 2015-09-21 at 23:07 -0500, Aaron Watry wrote: > > > On Thu, Sep 17, 2015 at 7:06 PM, Jan Vesely < > > > jan.ves...@rutgers.edu> > > > wrote: > > >

Re: [Piglit] [PATCH 2/2] cl: Change data types of char/short buffers in integer limits tests

2015-09-22 Thread Aaron Watry
On Tue, Sep 22, 2015 at 12:02 AM, Jan Vesely wrote: > On Mon, 2015-09-21 at 23:07 -0500, Aaron Watry wrote: > > On Thu, Sep 17, 2015 at 7:06 PM, Jan Vesely > > wrote: > > > > > On Thu, 2015-09-17 at 10:34 -0500, Aaron Watry wrote: > > > > On Wed,

Re: [Piglit] [PATCH 2/2] cl: Change data types of char/short buffers in integer limits tests

2015-09-21 Thread Jan Vesely
On Mon, 2015-09-21 at 23:07 -0500, Aaron Watry wrote: > On Thu, Sep 17, 2015 at 7:06 PM, Jan Vesely > wrote: > > > On Thu, 2015-09-17 at 10:34 -0500, Aaron Watry wrote: > > > On Wed, Sep 16, 2015 at 7:33 PM, Jan Vesely < > > > jan.ves...@rutgers.edu> > > > wrote: > > > >

Re: [Piglit] [PATCH 2/2] cl: Change data types of char/short buffers in integer limits tests

2015-09-21 Thread Aaron Watry
On Thu, Sep 17, 2015 at 7:06 PM, Jan Vesely wrote: > On Thu, 2015-09-17 at 10:34 -0500, Aaron Watry wrote: > > On Wed, Sep 16, 2015 at 7:33 PM, Jan Vesely > > wrote: > > > > > On Wed, 2015-09-16 at 17:18 -0500, Aaron Watry wrote: > > > > On Tue,

Re: [Piglit] [PATCH 2/2] cl: Change data types of char/short buffers in integer limits tests

2015-09-17 Thread Jan Vesely
On Thu, 2015-09-17 at 10:34 -0500, Aaron Watry wrote: > On Wed, Sep 16, 2015 at 7:33 PM, Jan Vesely > wrote: > > > On Wed, 2015-09-16 at 17:18 -0500, Aaron Watry wrote: > > > On Tue, Sep 15, 2015 at 9:42 AM, Jan Vesely < > > > jan.ves...@rutgers.edu> > > > wrote: > > > >

Re: [Piglit] [PATCH 2/2] cl: Change data types of char/short buffers in integer limits tests

2015-09-16 Thread Jan Vesely
On Wed, 2015-09-16 at 17:18 -0500, Aaron Watry wrote: > On Tue, Sep 15, 2015 at 9:42 AM, Jan Vesely > wrote: > > > > > > > On Tue, Sep 15, 2015 at 7:28 AM, Jan Vesely > > > > wrote: > > > > > On Thu, 2015-09-10 at 10:12 -0500, Aaron Watry

Re: [Piglit] [PATCH 2/2] cl: Change data types of char/short buffers in integer limits tests

2015-09-15 Thread Jan Vesely
On Thu, 2015-09-10 at 10:12 -0500, Aaron Watry wrote: > The char/short return buffers were declared as ints. > > Signed-off-by: Aaron Watry Reviewed-by: Jan Vesely For both patches. though, I agree with Serge that a spec reference would be nice. Jan

Re: [Piglit] [PATCH 2/2] cl: Change data types of char/short buffers in integer limits tests

2015-09-15 Thread Jan Vesely
On Tue, Sep 15, 2015 at 7:28 AM, Jan Vesely wrote: > On Thu, 2015-09-10 at 10:12 -0500, Aaron Watry wrote: > > The char/short return buffers were declared as ints. > > > > Signed-off-by: Aaron Watry > > Reviewed-by: Jan Vesely >

[Piglit] [PATCH 2/2] cl: Change data types of char/short buffers in integer limits tests

2015-09-10 Thread Aaron Watry
The char/short return buffers were declared as ints. Signed-off-by: Aaron Watry --- tests/cl/program/execute/int-definitions.cl | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/tests/cl/program/execute/int-definitions.cl