On Mon, 19 Jun 2023, Tamar Christina wrote:

> > -----Original Message-----
> > From: Richard Biener <rguent...@suse.de>
> > Sent: Monday, June 19, 2023 7:28 AM
> > To: gcc-patches@gcc.gnu.org
> > Cc: Tamar Christina <tamar.christ...@arm.com>
> > Subject: [PATCH] Remove -save-temps from tests using -flto
> > 
> > The following removes -save-temps that doesn't seem to have any good
> > reason from tests that also run with -flto added.  That can cause ltrans 
> > files to
> > race with other multilibs tested and I'm frequently seeing linker complaints
> > that the architecture doesn't match here.
> > 
> > I'm not sure whether the .ltrans.o files end up in a non gccN/ specific 
> > directory
> > or if we end up sharing the same dir for different multilibs (not sure if 
> > it's easily
> > possible to avoid that).
> > 
> > Parallel testing on x86_64-unknown-linux-gnu in progress.
> > 
> > Tamar, was there any reason to use -save-temps here?
> 
> At the time I was getting unresolved errors from these without it.
> But perhaps that's something to do with dejagnu versions?

I don't know.  Can you check if there's an issue on your side when
removing -save-temps?

Richard.

> Tamar
> 
> > 
> >     * gcc.dg/vect/vect-bic-bitmask-2.c: Remove -save-temps.
> >     * gcc.dg/vect/vect-bic-bitmask-3.c: Likewise.
> >     * gcc.dg/vect/vect-bic-bitmask-4.c: Likewise.
> >     * gcc.dg/vect/vect-bic-bitmask-5.c: Likewise.
> >     * gcc.dg/vect/vect-bic-bitmask-6.c: Likewise.
> >     * gcc.dg/vect/vect-bic-bitmask-8.c: Likewise.
> >     * gcc.dg/vect/vect-bic-bitmask-9.c: Likewise.
> >     * gcc.dg/vect/vect-bic-bitmask-10.c: Likewise.
> >     * gcc.dg/vect/vect-bic-bitmask-11.c: Likewise.
> > ---
> >  gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-10.c | 2 +-
> > gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-11.c | 2 +-
> > gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-2.c  | 2 +-
> > gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-3.c  | 2 +-
> > gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-4.c  | 2 +-
> > gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-5.c  | 2 +-
> > gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-6.c  | 2 +-
> > gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-8.c  | 2 +-
> > gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-9.c  | 2 +-
> >  9 files changed, 9 insertions(+), 9 deletions(-)
> > 
> > diff --git a/gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-10.c
> > b/gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-10.c
> > index e9ec9603af6..e6810433d70 100644
> > --- a/gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-10.c
> > +++ b/gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-10.c
> > @@ -1,6 +1,6 @@
> >  /* { dg-skip-if "missing optab for vectorization" { sparc*-*-* } } */
> >  /* { dg-do run } */
> > -/* { dg-additional-options "-O3 -save-temps -fdump-tree-dce -w" } */
> > +/* { dg-additional-options "-O3 -fdump-tree-dce -w" } */
> > 
> >  #include <stdint.h>
> > 
> > diff --git a/gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-11.c
> > b/gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-11.c
> > index 06c103d3885..f83078b5d51 100644
> > --- a/gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-11.c
> > +++ b/gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-11.c
> > @@ -1,6 +1,6 @@
> >  /* { dg-skip-if "missing optab for vectorization" { sparc*-*-* } } */
> >  /* { dg-do run } */
> > -/* { dg-additional-options "-O3 -save-temps -fdump-tree-dce -w" } */
> > +/* { dg-additional-options "-O3 -fdump-tree-dce -w" } */
> > 
> >  #include <stdint.h>
> > 
> > diff --git a/gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-2.c
> > b/gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-2.c
> > index 059bfb3ae62..e33a824df07 100644
> > --- a/gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-2.c
> > +++ b/gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-2.c
> > @@ -1,6 +1,6 @@
> >  /* { dg-skip-if "missing optab for vectorization" { sparc*-*-* } } */
> >  /* { dg-do run } */
> > -/* { dg-additional-options "-O3 -save-temps -fdump-tree-dce -w" } */
> > +/* { dg-additional-options "-O3 -fdump-tree-dce -w" } */
> > 
> >  #include <stdint.h>
> > 
> > diff --git a/gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-3.c
> > b/gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-3.c
> > index 059bfb3ae62..e33a824df07 100644
> > --- a/gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-3.c
> > +++ b/gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-3.c
> > @@ -1,6 +1,6 @@
> >  /* { dg-skip-if "missing optab for vectorization" { sparc*-*-* } } */
> >  /* { dg-do run } */
> > -/* { dg-additional-options "-O3 -save-temps -fdump-tree-dce -w" } */
> > +/* { dg-additional-options "-O3 -fdump-tree-dce -w" } */
> > 
> >  #include <stdint.h>
> > 
> > diff --git a/gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-4.c
> > b/gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-4.c
> > index 91b82fb5988..8895d5c263c 100644
> > --- a/gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-4.c
> > +++ b/gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-4.c
> > @@ -1,6 +1,6 @@
> >  /* { dg-skip-if "missing optab for vectorization" { sparc*-*-* } } */
> >  /* { dg-do run } */
> > -/* { dg-additional-options "-O3 -save-temps -fdump-tree-dce -w" } */
> > +/* { dg-additional-options "-O3 -fdump-tree-dce -w" } */
> > 
> >  #include <stdint.h>
> > 
> > diff --git a/gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-5.c
> > b/gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-5.c
> > index 59f339fb8c5..77d4deb633c 100644
> > --- a/gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-5.c
> > +++ b/gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-5.c
> > @@ -1,6 +1,6 @@
> >  /* { dg-skip-if "missing optab for vectorization" { sparc*-*-* } } */
> >  /* { dg-do run } */
> > -/* { dg-additional-options "-O3 -save-temps -fdump-tree-dce -w" } */
> > +/* { dg-additional-options "-O3 -fdump-tree-dce -w" } */
> > 
> >  #include <stdint.h>
> > 
> > diff --git a/gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-6.c
> > b/gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-6.c
> > index 84df8371be5..7930346a602 100644
> > --- a/gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-6.c
> > +++ b/gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-6.c
> > @@ -1,6 +1,6 @@
> >  /* { dg-skip-if "missing optab for vectorization" { sparc*-*-* } } */
> >  /* { dg-do run } */
> > -/* { dg-additional-options "-O3 -save-temps -fdump-tree-dce -w" } */
> > +/* { dg-additional-options "-O3 -fdump-tree-dce -w" } */
> > 
> >  #include <stdint.h>
> > 
> > diff --git a/gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-8.c
> > b/gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-8.c
> > index b3e0b5ba92f..1af7104a37d 100644
> > --- a/gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-8.c
> > +++ b/gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-8.c
> > @@ -1,6 +1,6 @@
> >  /* { dg-skip-if "missing optab for vectorization" { sparc*-*-* } } */
> >  /* { dg-do run } */
> > -/* { dg-additional-options "-O3 -save-temps -fdump-tree-dce -w" } */
> > +/* { dg-additional-options "-O3 -fdump-tree-dce -w" } */
> > 
> >  #include <stdint.h>
> > 
> > diff --git a/gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-9.c
> > b/gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-9.c
> > index 5fcde11ad4e..f0a75604bcb 100644
> > --- a/gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-9.c
> > +++ b/gcc/testsuite/gcc.dg/vect/vect-bic-bitmask-9.c
> > @@ -1,6 +1,6 @@
> >  /* { dg-skip-if "missing optab for vectorization" { sparc*-*-* } } */
> >  /* { dg-do run } */
> > -/* { dg-additional-options "-O3 -save-temps -fdump-tree-dce -w" } */
> > +/* { dg-additional-options "-O3 -fdump-tree-dce -w" } */
> > 
> >  #include <stdint.h>
> > 
> > --
> > 2.35.3
> 

-- 
Richard Biener <rguent...@suse.de>
SUSE Software Solutions Germany GmbH, Frankenstrasse 146, 90461 Nuernberg,
Germany; GF: Ivo Totev, Andrew Myers, Andrew McDonald, Boudien Moerman;
HRB 36809 (AG Nuernberg)

Reply via email to