Hi!
On Wed, Feb 21, 2018 at 11:10:25AM -0800, Carl Love wrote:
> Per discussions with Segher, we felt it would be best to move the
> vec_float2 test to a Power 8 test as it is only defined for Power 8 and
> beyond. In doing this, I found that compiling builtins-3-runnable.c
> with -mcpu=power7 then generated an ICE for vec_signed2 and
> vec_unsigned2. It seems that once gcc saw the unsupported vec_float2
> it exited with -mcpu=power7 it exited before seeing the vec_signed2 and
> vec_unsigned2 builtins. The vec_signed2 and vec_unsigned2 are also
> only defined for Power 8 and beyond.
>
> This patch moves the three power 8 builtins to a new test file
> builtins-3-runnable-p8.c. It also fixed the ICE by restricting the
> vec_signed2 and vec_unsigned2 builtins to Power 8. The builtins-3-
> runnable.c dg settings are then set to enable the test to run on Power
> 7.
> --- a/gcc/testsuite/gcc.target/powerpc/builtins-3-runnable.c
> +++ b/gcc/testsuite/gcc.target/powerpc/builtins-3-runnable.c
> @@ -1,7 +1,6 @@
> /* { dg-do run { target { p8vector_hw } } } */
> /* { dg-require-effective-target vsx_hw } */
> -/* { dg-options "-O2 -mvsx -mcpu=power8" } */
> -/* { dg-skip-if "do not override -mcpu" { powerpc*-*-* } { "-mcpu=*" } {
> "-mcpu=power8" } } */
> +/* { dg-options "-maltivec -mvsx" } */
I think you should change the p8vector_hw here? To vsx_hw?
Okay for trunk with that change (please retest this test, of course).
Thanks!
Segher
