On Tue, 2020-06-30 at 18:39 -0500, Segher Boessenkool wrote:
> Hi!
> 
> On Tue, Jun 30, 2020 at 12:57:45PM -0500, will schmidt wrote:
> >   Add support for the vmsumudm instruction and tie it into the
> > vec_msum
> >   built-in to support the variants of that built-in using vector
> >  _int128 parameters.
> > 2020-06-18  Will Schmidt  <will_schm...@vnet.ibm.com>
> > 
> >         * config/rs6000/altivec.h (vec_vmsumudm): New define.
> >         * config/rs6000/altivec.md (UNSPEC_VMSUMUDM): New unspec.
> >         (altivec_vmsumudm): New define_insn.
> >         * config/rs6000/rs6000-builtin.def (altivec_vmsumudm): New
> >         BU_ALTIVEC_3 entry. (vmsumudm): New BU_ALTIVEC_OVERLOAD_3
> >         entry.
> 
> No line break before (vmsumudm) please.
> 
> >         * config/rs6000/rs6000-call.c
> > (altivec_overloaded_builtins):
> >         Add entries for ALTIVEC_BUILTIN_VMSUMUDM variants of
> > vec_msum.
> 
> Tha patch is okay for trunk with that (and some int128 selector in
> the
> testcases that need one).  Thanks!

Thanks for the review, etc.

OK for backports too? (after baking on trunk for a bit).

thanks
-Will

> 
> 
> Segher

Reply via email to