Re: [patch, testsuite] Fix PR tree-optimization/39422: vectorizer tests failure on SPU

[Richard Guenther <richard dot guenther at gmail dot com>]

On Tue, Mar 10, 2009 at 3:14 PM, Ira Rosen <IRAR@il.ibm.com> wrote:
>
> Hi,
>
> This patch fixes two SPU vectorizer tests that were missed by this patch
> http://gcc.gnu.org/ml/gcc-patches/2008-08/msg01160.html. The change is to
> move constant arrays with static initialization to global memory.
>
> Tested on Cell SPU. O.K. for mainline?

Ok.  Does it still test the same thing though?  That is, is the alignment
inside the function lower (or higher) than 16?

Thanks,
Richard.

> Thanks,
> Ira
>
> testsuite/ChangeLog:
>
> ? ? ?PR tree-optimization/39422
> ? ? ?* gcc.dg/vect/costmodel/spu/costmodel-vect-76b.c: Move
> ? ? ?constant array with static initialization to global memory.
> ? ? ?* gcc.dg/vect/costmodel/spu/costmodel-vect-76c.c: Likewise.
>
> Index: testsuite/gcc.dg/vect/costmodel/spu/costmodel-vect-76b.c
> ===================================================================
> --- testsuite/gcc.dg/vect/costmodel/spu/costmodel-vect-76b.c ? ?(revision
> 144745)
> +++ testsuite/gcc.dg/vect/costmodel/spu/costmodel-vect-76b.c ? ?(working
> copy)
> @@ -11,12 +11,12 @@
> ? ?more involved than just an ssa_name. ?*/
>
> ?int ib[N+OFF] __attribute__ ((__aligned__(16))) = {0, 1, 3, 5, 7, 11, 13,
> 17, 0, 2, 6, 10, 0, 1, 3, 5, 7, 11, 13, 17};
> +int ic[N+OFF] = {0, 1, 3, 5, 7, 11, 13, 17, 0, 2, 6, 10, 0, 1, 3, 5, 7,
> 11, 13, 17};
>
> ?int main1 (int *pib)
> ?{
> ? int i;
> ? int ia[N+OFF];
> - ?int ic[N+OFF] = {0, 1, 3, 5, 7, 11, 13, 17, 0, 2, 6, 10, 0, 1, 3, 5, 7,
> 11, 13, 17};
>
> ? for (i = OFF; i < N; i++)
> ? ? {
> Index: testsuite/gcc.dg/vect/costmodel/spu/costmodel-vect-76c.c
> ===================================================================,
> --- testsuite/gcc.dg/vect/costmodel/spu/costmodel-vect-76c.c ? ?(revision
> 144745),
> +++ testsuite/gcc.dg/vect/costmodel/spu/costmodel-vect-76c.c ? ?(working
> copy)
> @@ -11,12 +11,12 @@
> ? ?more involved than just an ssa_name. ?*/
>
> ?int ib[N+OFF] __attribute__ ((__aligned__(16))) = {0, 1, 3, 5, 7, 11, 13,
> 17, 0, 2, 6, 10};
> +int ic[N+OFF] = {0, 1, 3, 5, 7, 11, 13, 17, 0, 2, 6, 10};
>
> ?int main1 (int *pib)
> ?{
> ? int i;
> ? int ia[N+OFF];
> - ?int ic[N+OFF] = {0, 1, 3, 5, 7, 11, 13, 17, 0, 2, 6, 10};
>
> ? for (i = OFF; i < N; i++)
> ? ? {
>
>