On Wed, Jan 17, 2018 at 09:51:54AM -0600, Will Schmidt wrote:
> On Tue, 2018-01-16 at 16:34 -0600, Segher Boessenkool wrote:
> > Hi!
> > On Tue, Jan 16, 2018 at 01:39:28PM -0600, Will Schmidt wrote:
> > > --- /dev/null
> > > +++ b/gcc/testsuite/gcc.target/powerpc/builtins-1-be-folded.c
> > > @@ -0,0 +1,11 @@
> > > +/* { dg-do compile { target { powerpc-*-* } } } */
> >
> > Do you want powerpc*-*-*? That is default in gcc.target/powerpc; dg-do
> > compile is default, too, so you can either say
> >
> > /* { dg-do compile } */
> >
> > or nothing at all, to taste.
> >
> > But it looks like you want to restrict to BE? We still don't have a
> > dejagnu thingy for that; you could put some #ifdef around it all (there
> > are some examples in other testcases). Not ideal, but works.
>
> Just want to ensure continuing coverage. :-) This test in particular
> is a copy/paste + tweak of an existing test, which tries to limit itself
> to BE, there is an LE counterpart.
powerpc-*-* means those compilers that were configured for a 32-bit BE
default target. Which we do not usually have these days. It also doesn't
say much about what target the test is running for.
> My regression test results suggest that the addition of the
> -mno-fold-gimple option to the existing testcases appears to have
> uncovered an ICE, so pausing for the moment...
Good luck :-) If you are reasonably certain the bug is not in your patch
(but pre-existing), please do commit the patch.
Segher
