Hi Carl,

On Mon, Apr 30, 2018 at 09:52:27AM -0700, Carl Love wrote:
> gcc/testsuite/ChangeLog:
> 
> 2018-04-30  Carl Love  <c...@us.ibm.com>
>       * gcc.target/powerpc/vsx-vector-6.h (foo): Add test for vec_max,
>       vec_trunc.
>       * gcc.target/powerpc/vsx-vector-6-le.c (dg-final): Update xvcmpeqdp,
>       xvcmpgtdp, xvcmpgedp counts.

> @@ -65,8 +70,6 @@ void foo (vector double *out, vector double *in, vector 
> long *p_l, vector bool l
>    *out++ = vec_or (inb, in0);
>    *out++ = vec_perm (in0, in1, uc);
>    *out++ = vec_rint (in0);
> -  *out++ = vec_sel (in0, in1, inl);
> -  *out++ = vec_sel (in0, in1, inb);
>    *out++ = vec_sub (in0, in1);
>    *out++ = vec_sqrt (in0);
>    *out++ = vec_trunc (in0);

Why does the patch remove these two vec_sel?  If that is wanted, the
changelog should mention this.

Looks fine otherwise.


Segher

Reply via email to