On Wed, Jul 08, 2020 at 10:55:59AM -0500, will schmidt wrote:
> On Tue, 2020-06-30 at 18:39 -0500, Segher Boessenkool wrote:
> > 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).

Yes please.  Do it in time before the RC?  Thanks!


Segher

Reply via email to