Hi!

On Thu, Jan 28, 2021 at 02:40:25AM -0600, Xionghu Luo wrote:
> Move common functions to header file for cleanup.
> 
> gcc/testsuite/ChangeLog:
> 
> 2021-01-27  Xionghu Luo  <luo...@linux.ibm.com>
> 
>       * gcc.target/powerpc/pr79251.p8.c: Move definition to ...
>       * gcc.target/powerpc/pr79251.h: ...this.
>       * gcc.target/powerpc/pr79251.p9.c: Likewise.
>       * gcc.target/powerpc/pr79251-run.c: Rename to...
>       * gcc.target/powerpc/pr79251-run.p8.c: ...this.
>       * gcc.target/powerpc/pr79251-run.p9.c: New test.

> --- a/gcc/testsuite/gcc.target/powerpc/pr79251.p8.c
> +++ b/gcc/testsuite/gcc.target/powerpc/pr79251.p8.c
> @@ -6,8 +6,6 @@
>  #include <altivec.h>
>  #include "pr79251.h"
>  
> -TEST_VEC_INSERT_ALL (test)

The changelog does not mention this (unless that is what "move
definition" means -- please improve the changelog a bit).

But the patch is fine as far as I can see.  Okay for trunk.  Thanks!


Segher

Reply via email to