On Tue, 2017-07-25 at 06:19 -0500, Segher Boessenkool wrote:
> Hi!
>
> On Mon, Jul 24, 2017 at 01:51:20PM -0700, Carl Love wrote:
> > On Fri, 2017-07-14 at 15:58 -0500, Segher Boessenkool wrote:
> > > On Fri, Jul 14, 2017 at 01:20:32PM -0700, Carl Love wrote:
> > > > --- /dev/null
> > > > +++ b/gcc/testsuite/gcc.target/powerpc/builtins-4-p9-runnable.c
> > > > @@ -0,0 +1,95 @@
> > > > +/* { dg-do run { target { powerpc64*-*-* && { lp64 && p9vector_hw } }
> > > > } } */
> > >
> > > powerpc*-*-* instead.
> > >
> > > > +/* { dg-skip-if "do not override -mcpu" { powerpc*-*-* } { "-mcpu=*" }
> > > > { "-mcpu=power9" } } */
> > > > +/* { dg-options "-mcpu=power9 -O2 -mupper-regs-di" } */
> > >
> > > Does this need to be restricted to 64-bit? Please test.
>
> I mean: do you need the "lp64". So, does
>
> /* { dg-do run { target { powerpc*-*-* && { p9vector_hw } } } } */
>
> work?
/* { dg-do run { target { powerpc*-*-* && { p9vector_hw } } } } */
/* { dg-skip-if "do not override -mcpu" { powerpc*-*-* } { "-mcpu=*" } {
"-mcpu=power9" } } */
/* { dg-options "-mcpu=power9 -O2 -mupper-regs-di" } */
m64 2 test passes
m32 1 unsupported test
I looked in gcc/testsuite/gcc/gcc.log and found the following message
about the -m32
cc1: error: -m32 not supported in this configuration^M
compiler exited with status 1
testcase /home/carll/GCC/gcc-patch-testing/gcc/testsuite/gcc.target/powerpc/pow\
erpc.exp completed in 1 seconds
=== gcc Summary for unix/-m32 ===
# of unsupported tests 1
=== gcc Summary ===
# of expected passes 2
# of unsupported tests 1
> Or maybe it is the -mupper-regs-di? That option is
> has been deleted now btw.
>
So, next I tried removing the -mupper-regs-di option.
/* { dg-do run { target { powerpc*-*-* && { p9vector_hw } } } } */
/* { dg-skip-if "do not override -mcpu" { powerpc*-*-* } { "-mcpu=*" } {
"-mcpu=power9" } } */
/* { dg-options "-mcpu=power9 -O2 " } */
With the same results as above. Note, I am running on
perch.aus.stglabs.ibm.com which
is a Power 9 system. Is -m32 not supported on Power 9?
So it looks like we do not need the lp64 or -mupper-regs-di options as we get
the same results
either way.
Carl
