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