This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix powerpc ICE with __builtin_vec_ld on an array (PR target/82112, take 2)
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Segher Boessenkool <segher at kernel dot crashing dot org>
- Cc: David Edelsohn <dje dot gcc at gmail dot com>, gcc-patches at gcc dot gnu dot org, wschmidt at linux dot vnet dot ibm dot com
- Date: Tue, 12 Sep 2017 17:03:54 +0200
- Subject: Re: [PATCH] Fix powerpc ICE with __builtin_vec_ld on an array (PR target/82112, take 2)
- Authentication-results: sourceware.org; auth=none
- Authentication-results: ext-mx06.extmail.prod.ext.phx2.redhat.com; dmarc=none (p=none dis=none) header.from=redhat.com
- Authentication-results: ext-mx06.extmail.prod.ext.phx2.redhat.com; spf=fail smtp.mailfrom=jakub at redhat dot com
- Dmarc-filter: OpenDMARC Filter v1.3.2 mx1.redhat.com 1D699356E8
- References: <20170907084030.GS2323@tucnak> <20170912142548.GI1701@tucnak> <20170912150044.GE8421@gate.crashing.org>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Tue, Sep 12, 2017 at 10:00:44AM -0500, Segher Boessenkool wrote:
> > --- gcc/testsuite/g++.dg/ext/altivec-18.C.jj 2017-09-08 09:15:20.593774717 +0200
> > +++ gcc/testsuite/g++.dg/ext/altivec-18.C 2017-09-08 09:15:20.593774717 +0200
> > @@ -0,0 +1,14 @@
> > +// PR target/82112
> > +// { dg-do compile { target powerpc*-*-* } }
> > +// { dg-require-effective-target powerpc_altivec_ok }
> > +// { dg-options "-save-temps -maltivec" }
>
> What is this -save-temps for? Just a leftover?
It is not really needed, I've just copied/pasted those 3 lines
from some other testcase (which probably used it uselessly as well).
-save-temps can make sense for dg-do link/run testcases where somebody
also wants to dg-final scan-assembler, or for the very rare case when
actually the -save-temps behavior needs to be tested (using separate
preprocessor etc.). Will fix.
Jakub