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

Jan Vesely jan.vesely at rutgers.edu
Tue Sep 15 07:42:31 PDT 2015


On Tue, Sep 15, 2015 at 7:28 AM, Jan Vesely <jan.vesely at rutgers.edu> 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 <awatry at gmail.com>
>
> Reviewed-by: Jan Vesely <jan.vesely at rutgers.edu>
> For both patches.
>
> though, I agree with Serge that a spec reference would be nice.
>

PS: don't we need to test this for *_MAX too? I'd expect at least char and
short to have the same problem.

>
> Jan
>
> > ---
> >  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
> > b/tests/cl/program/execute/int-definitions.cl
> > index 3d8ee63..a438fe4 100644
> > --- a/tests/cl/program/execute/int-definitions.cl
> > +++ b/tests/cl/program/execute/int-definitions.cl
> > @@ -12,12 +12,12 @@ global_size: 1 0 0
> >  [test]
> >  name: Char Definitions
> >  kernel_name: test_char
> > -arg_out: 0 buffer int[6] 8 127 -128 127 -128 255
> > +arg_out: 0 buffer char[6] 8 127 -128 127 -128 255
> >
> >  [test]
> >  name: Short Definitions
> >  kernel_name: test_short
> > -arg_out: 0 buffer int[3] 32767 -32768 65535
> > +arg_out: 0 buffer short[3] 32767 -32768 65535
> >
> >  [test]
> >  name: Int Definitions
> > @@ -32,7 +32,7 @@ arg_out: 0 buffer long[3] 9223372036854775807 \
> >                            18446744073709551615
> >  !*/
> >
> > -kernel void test_char(global int* out) {
> > +kernel void test_char(global char* out) {
> >    int i = 0;
> >    out[i++] = CHAR_BIT;
> >    out[i++] = CHAR_MAX;
> > @@ -42,7 +42,7 @@ kernel void test_char(global int* out) {
> >    out[i++] = UCHAR_MAX;
> >  }
> >
> > -kernel void test_short(global int* out) {
> > +kernel void test_short(global short* out) {
> >    int i = 0;
> >    out[i++] = SHRT_MAX;
> >    out[i++] = (SHRT_MIN - (short2)(0)).s0;
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/piglit/attachments/20150915/b02458a1/attachment.html>


More information about the Piglit mailing list