Patchwork Fix the tests gcc.dg/vect/vect-8[23]_64.c

login
register
mail settings
Submitter Dominique Dhumieres
Date Oct. 23, 2012, 1:52 p.m.
Message ID <20121023135237.0657B3BABA@mailhost.lps.ens.fr>
Download mbox | patch
Permalink /patch/193498/
State New
Headers show

Comments

Dominique Dhumieres - Oct. 23, 2012, 1:52 p.m.
Following the changes in [PATCH] Add option for dumping to stderr (issue6190057)
the tests gcc.dg/vect/vect-8[23]_64.c fails on powerpc*-*-*.
This patch adjust the dump files and has been tested on powerpc-apple-darwin9.

If approved could someone commit it for me (no write access).

Note that these tests use both dg-do run and dg-do compile which is not
supported (see http://gcc.gnu.org/ml/gcc/2012-10/msg00226.html and
the rest of the thread).

TIA

Dominique

gcc/testsuite/ChangeLog
2012-10-23  Dominique d'Humieres  <dominiq@lps.ens.fr>

	* gcc.dg/vect/vect-82_64.c: Adjust the dump file.
	* gcc.dg/vect/vect-83_64.c: Likewise.
Sharad Singhai - Oct. 23, 2012, 6:21 p.m.
+cc richard.guenther@gmail.com

If it is approved, I will be happy to commit it for you.

Thanks,
Sharad
Sharad


On Tue, Oct 23, 2012 at 6:52 AM, Dominique Dhumieres <dominiq@lps.ens.fr> wrote:
> Following the changes in [PATCH] Add option for dumping to stderr (issue6190057)
> the tests gcc.dg/vect/vect-8[23]_64.c fails on powerpc*-*-*.
> This patch adjust the dump files and has been tested on powerpc-apple-darwin9.
>
> If approved could someone commit it for me (no write access).
>
> Note that these tests use both dg-do run and dg-do compile which is not
> supported (see http://gcc.gnu.org/ml/gcc/2012-10/msg00226.html and
> the rest of the thread).
>
> TIA
>
> Dominique
>
> gcc/testsuite/ChangeLog
> 2012-10-23  Dominique d'Humieres  <dominiq@lps.ens.fr>
>
>         * gcc.dg/vect/vect-82_64.c: Adjust the dump file.
>         * gcc.dg/vect/vect-83_64.c: Likewise.
>
> diff -up gcc/testsuite/gcc.dg/vect/vect-82_64.c ../work/gcc/testsuite/gcc.dg/vect/vect-82_64.c
> --- gcc/testsuite/gcc.dg/vect/vect-82_64.c      2007-11-21 20:18:48.000000000 +0100
> +++ ../work/gcc/testsuite/gcc.dg/vect/vect-82_64.c      2012-10-08 13:52:25.000000000 +0200
> @@ -1,6 +1,6 @@
>  /* { dg-do run { target { { powerpc*-*-* && lp64 } && powerpc_altivec_ok } } } */
>  /* { dg-do compile { target { { powerpc*-*-* && ilp32 } && powerpc_altivec_ok } } } */
> -/* { dg-options "-O2 -ftree-vectorize -mpowerpc64 -fdump-tree-vect-stats -maltivec" } */
> +/* { dg-options "-O2 -ftree-vectorize -mpowerpc64 -fdump-tree-vect-details -maltivec" } */
>
>  #include <stdarg.h>
>  #include "tree-vect.h"
> diff -up gcc/testsuite/gcc.dg/vect/vect-83_64.c ../work/gcc/testsuite/gcc.dg/vect/vect-83_64.c
> --- gcc/testsuite/gcc.dg/vect/vect-83_64.c      2007-11-21 20:18:48.000000000 +0100
> +++ ../work/gcc/testsuite/gcc.dg/vect/vect-83_64.c      2012-10-08 13:52:42.000000000 +0200
> @@ -1,6 +1,6 @@
>  /* { dg-do run { target { { powerpc*-*-* && lp64 } && powerpc_altivec_ok } } } */
>  /* { dg-do compile { target { { powerpc*-*-* && ilp32 } && powerpc_altivec_ok } } } */
> -/* { dg-options "-O2 -ftree-vectorize -mpowerpc64 -fdump-tree-vect-stats -maltivec" } */
> +/* { dg-options "-O2 -ftree-vectorize -mpowerpc64 -fdump-tree-vect-details -maltivec" } */
>
>  #include <stdarg.h>
>  #include "tree-vect.h"
Mike Stump - Oct. 23, 2012, 9:46 p.m.
On Oct 23, 2012, at 6:52 AM, Dominique Dhumieres <dominiq@lps.ens.fr> wrote:
> Following the changes in [PATCH] Add option for dumping to stderr (issue6190057)
> the tests gcc.dg/vect/vect-8[23]_64.c fails on powerpc*-*-*.
> This patch adjust the dump files and has been tested on powerpc-apple-darwin9.

Ok.

> gcc/testsuite/ChangeLog
> 2012-10-23  Dominique d'Humieres  <dominiq@lps.ens.fr>
> 
> 	* gcc.dg/vect/vect-82_64.c: Adjust the dump file.
> 	* gcc.dg/vect/vect-83_64.c: Likewise.
Sharad Singhai - Oct. 24, 2012, 12:17 a.m.
Committed in r192750.

Thanks,
Sharad

On Tue, Oct 23, 2012 at 2:46 PM, Mike Stump <mikestump@comcast.net> wrote:
> On Oct 23, 2012, at 6:52 AM, Dominique Dhumieres <dominiq@lps.ens.fr> wrote:
>> Following the changes in [PATCH] Add option for dumping to stderr (issue6190057)
>> the tests gcc.dg/vect/vect-8[23]_64.c fails on powerpc*-*-*.
>> This patch adjust the dump files and has been tested on powerpc-apple-darwin9.
>
> Ok.
>
>> gcc/testsuite/ChangeLog
>> 2012-10-23  Dominique d'Humieres  <dominiq@lps.ens.fr>
>>
>>       * gcc.dg/vect/vect-82_64.c: Adjust the dump file.
>>       * gcc.dg/vect/vect-83_64.c: Likewise.
>
Dominique Dhumieres - Oct. 24, 2012, 8:53 p.m.
Mike, Sharad,

> Committed in r192750.

Thanks for the review and the commit.

Dominique

Patch

diff -up gcc/testsuite/gcc.dg/vect/vect-82_64.c ../work/gcc/testsuite/gcc.dg/vect/vect-82_64.c
--- gcc/testsuite/gcc.dg/vect/vect-82_64.c	2007-11-21 20:18:48.000000000 +0100
+++ ../work/gcc/testsuite/gcc.dg/vect/vect-82_64.c	2012-10-08 13:52:25.000000000 +0200
@@ -1,6 +1,6 @@ 
 /* { dg-do run { target { { powerpc*-*-* && lp64 } && powerpc_altivec_ok } } } */
 /* { dg-do compile { target { { powerpc*-*-* && ilp32 } && powerpc_altivec_ok } } } */
-/* { dg-options "-O2 -ftree-vectorize -mpowerpc64 -fdump-tree-vect-stats -maltivec" } */
+/* { dg-options "-O2 -ftree-vectorize -mpowerpc64 -fdump-tree-vect-details -maltivec" } */
 
 #include <stdarg.h>
 #include "tree-vect.h"
diff -up gcc/testsuite/gcc.dg/vect/vect-83_64.c ../work/gcc/testsuite/gcc.dg/vect/vect-83_64.c
--- gcc/testsuite/gcc.dg/vect/vect-83_64.c	2007-11-21 20:18:48.000000000 +0100
+++ ../work/gcc/testsuite/gcc.dg/vect/vect-83_64.c	2012-10-08 13:52:42.000000000 +0200
@@ -1,6 +1,6 @@ 
 /* { dg-do run { target { { powerpc*-*-* && lp64 } && powerpc_altivec_ok } } } */
 /* { dg-do compile { target { { powerpc*-*-* && ilp32 } && powerpc_altivec_ok } } } */
-/* { dg-options "-O2 -ftree-vectorize -mpowerpc64 -fdump-tree-vect-stats -maltivec" } */
+/* { dg-options "-O2 -ftree-vectorize -mpowerpc64 -fdump-tree-vect-details -maltivec" } */
 
 #include <stdarg.h>
 #include "tree-vect.h"