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

[Michael Zolotukhin <michael dot v dot zolotukhin at gmail dot com>]

> 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).

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
Description: Binary data