Re: [Patch] Increase array sizes in vect-tests to enable 256-bit vectorization

[Ira Rosen <IRAR at il dot ibm dot com>]

Michael Zolotukhin <michael.v.zolotukhin@gmail.com> wrote on 02/12/2011
08:11:41 PM:
>
> > Please don't change initial values to 0, we want to check that
everything
> > works fine for non-zeros as well.
> > There are several other occasions in the patch.
>
> Please check the update patch (attached).

This is ok with me.

Thanks,
Ira

>
> On 2 December 2011 20:49, Ira Rosen <IRAR@il.ibm.com> wrote:
> >
> > gcc-patches-owner@gcc.gnu.org wrote on 02/12/2011 06:23:25 PM:
> >
> >> Hi,
> >>
> >> This patch increases array sizes in tests from vect.exp suite, thus
> >> enabling 256-bit vectorization where it's available.
> >>
> >> Ok for trunk?
> >
> > --- a/gcc/testsuite/gcc.dg/vect/slp-24.c
> > +++ b/gcc/testsuite/gcc.dg/vect/slp-24.c
> > ...
> > @@ -13,14 +12,17 @@ typedef struct {
> > ? ?unsigned char d;
> > ?} s;
> >
> > -unsigned char ub[N*2] =
> > {1,3,6,9,12,15,18,21,24,27,30,33,36,39,42,45,1,3,6,9,12,15,18,21,
> 24,27,30,33,36,39,42,45};
> > -unsigned char uc[N] = {1,1,2,3,4,5,6,7,8,9,10,11,12,13,14,15};
> > +unsigned char ub[N*2];
> > +unsigned char uc[N];
> > +
> > +volatile int y = 0;
> > +unsigned char check_diff = 0;
> >
> > ?void
> > ?main1 (unsigned char x, unsigned char max_result, unsigned char
> > min_result, s *arr)
> > ?{
> > ? int i;
> > - ?unsigned char udiff = 2;
> > + ?unsigned char udiff = 0;
> >
> > Please don't change initial values to 0, we want to check that
everything
> > works fine for non-zeros as well.
> > There are several other occasions in the patch.
> >
> > Thanks,
> > Ira
> >
> >>
> >> Changelog:
> >> 2011-12-02 ?Michael Zolotukhin ?<michael.v.zolotukhin@intel.com>
> >>
> >> ? ?* gcc.dg/vect/slp-13.c: Increase array size, add initialization.
> >> ? ?* gcc.dg/vect/slp-24.c: Ditto.
> >> ? ?* gcc.dg/vect/slp-3.c: Likewise and fix scans.
> >> ? ?* gcc.dg/vect/slp-34.c: Ditto.
> >> ? ?* gcc.dg/vect/slp-4.c: Ditto.
> >> ? ?* gcc.dg/vect/slp-cond-2.c: Ditto.
> >> ? ?* gcc.dg/vect/slp-multitypes-11.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-1.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-10.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-105.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-112.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-15.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-2.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-31.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-32.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-33.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-34.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-35.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-36.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-6.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-73.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-74.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-75.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-76.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-80.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-85.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-89.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-97.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-98.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-all.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-double-reduc-6.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-iv-8.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-iv-8a.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-outer-1.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-outer-1a.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-outer-1b.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-outer-2.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-outer-2a.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-outer-2c.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-outer-3.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-outer-3a.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-outer-4a.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-outer-4b.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-outer-4c.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-outer-4d.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-outer-4m.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-outer-fir-lb.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-outer-fir.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-over-widen-1.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-over-widen-2.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-over-widen-3.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-over-widen-4.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-reduc-1char.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-reduc-2char.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-reduc-pattern-1b.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-reduc-pattern-1c.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-reduc-pattern-2b.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-shift-2.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-strided-a-u8-i8-gap2.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-strided-a-u8-i8-gap7.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-strided-u8-i8-gap2.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-strided-u8-i8-gap4.c: Ditto.
> >> ? ?* gcc.dg/vect/vect-strided-u8-i8-gap7.c: Ditto.
> >>
> >> --
> >> ---
> >> Best regards,
> >> Michael V. Zolotukhin,
> >> Software Engineer
> >> Intel Corporation.
> >> [attachment "vect_tests.patch" deleted by Ira Rosen/Haifa/IBM]
> >
>
>
>
> --
> ---
> Best regards,
> Michael V. Zolotukhin,
> Software Engineer
> Intel Corporation.
> [attachment "vect_tests-2.patch" deleted by Ira Rosen/Haifa/IBM]