Hi!

On Thu, Jun 13, 2024 at 11:32:58AM -0700, Carl Love wrote:
> The test gcc/testsuite/gcc.target/powerpc/altivec-2-runnable.c is supposed to 
> be a runnable test
> to verify the execution of the vec_unpackl and vec_unpackh built-ins.  The 
> dg-do command is set to
> compile not run.  This patch fixes the dg-do command argument.
> 
> The patch has been verified on a P10.  The test runs without errors.

> rs6000, altivec-2-runnable.c should be a runnable test
> 
> The test case has "dg-do compile" set not "dg-do run" for a runnable
> test.  This patch changes the dg-do command argument to run.
> 
> gcc/testsuite/ChangeLog:gcc/testsuite/ChangeLog:
>       * gcc.target/powerpc/altivec-2-runnable.c: Change dg-do
>       argument to run.


> --- a/gcc/testsuite/gcc.target/powerpc/altivec-2-runnable.c
> +++ b/gcc/testsuite/gcc.target/powerpc/altivec-2-runnable.c
> @@ -1,4 +1,4 @@
> -/* { dg-do compile { target powerpc*-*-* } } */
> +/* { dg-do run { target powerpc*-*-* } } */
>  /* { dg-options "-mvsx" } */
>  /* { dg-additional-options "-mdejagnu-cpu=power8" { target { ! has_arch_pwr8 
> } } } */
>  /* { dg-require-effective-target powerpc_vsx } */

Everything in gcc.target/powerpc/ is tested for "target powerpc*-*-*"
already, so you could remove that target clause even (after testing of
course :-) )

Okay for trunk with or without that extra tweak.  Thank you!


Segher

Reply via email to