Hi Carl,

On Wed, Nov 08, 2017 at 01:21:46PM -0800, Carl Love wrote:
> 2017-11-08  Carl Love  <c...@us.ibm.com>
> 
>       * config/rs6000/rs6000-c.c (ALTIVEC_BUILTIN_VEC_SUMS): Add macro 
> expansion.

This line is too long (79 or 80 chars).

> +(define_expand "altivec_vsumsws_be"
> +  [(use (match_operand:V4SI 0 "register_operand"))
> +   (use (match_operand:V4SI 1 "register_operand"))
> +   (use (match_operand:V4SI 2 "register_operand"))]
> +  "TARGET_ALTIVEC"
> +{
> +  emit_insn (gen_altivec_vsumsws_direct (operands[0], operands[1],
> +                                      operands[2]));
> +  DONE;
> +})

Can't you call altivec_vsumsws_direct from the builtin directly?  That
would be prefered.

Okay for trunk either way.  Thanks!


Segher

Reply via email to