On 10/12/2012 08:05 AM, Joe Seymour wrote:
> I'm observing vect/pr48765.c fail for non 64-bit PowerPC targets:
>
>> gcc/testsuite/gcc.dg/vect/pr48765.c:1:0: error: -m64 not supported in this
> configuration
>
> This patch restricts the test to 64-bit PowerPC targets. I don't have commit
> access, so if this OK perhaps someone could commit it for me?
>
> Thanks,
>
> 2012-10-12 Joe Seymour <jseym...@codesourcery.com>
>
> * gcc.dg/vect/pr48765.c: Restrict to 64-bit PowerPC targets.
>
> Index: gcc/testsuite/gcc.dg/vect/pr48765.c
> ===================================================================
> --- gcc/testsuite/gcc.dg/vect/pr48765.c (revision 192402)
> +++ gcc/testsuite/gcc.dg/vect/pr48765.c (working copy)
> @@ -1,4 +1,4 @@
> -/* { dg-do compile { target powerpc*-*-* } } */
> +/* { dg-do compile { target powerpc64 } } */
> /* { dg-options "-m64 -O3 -mcpu=power6" } */
>
> enum reg_class
>
No, the test should not add -m64, and -mcpu=power6 might conflict
with multilib options. I proposed a different fix for this test in
<http://gcc.gnu.org/ml/gcc-patches/2012-09/msg01320.html> and then
didn't follow up on it.
I've copied Ira who added the test to see what she thinks of my
proposed change, and David as the Power maintainer.
Janis
