Hi! On Wed, Jun 24, 2020 at 02:28:00PM -0500, Peter Bergner via Gcc-patches wrote: > On 6/18/20 3:42 PM, Peter Bergner wrote: > > POWER ISA 3.1 added new Matrix-Multiply Assist (MMA) instructions. > > The following patch set adds support for generating these instructions > > through built-in functions which are enabled with the -mmma option. > > > > The patch1 and patch1+patch2+patch3 have been bootstrapped and regtested on > > powerpc64le-linux with no regressions. In addition, patch1+patch2+patch3 > > has been bootstrapped and regtested on powerpc64-linux (BE), also without > > regressions. I'll note that I split the testsuite changes into their own > > patch for review purposes, but I plan on committing patch2 and patch3 > > together. > > > > Changes since v1: > > Patch 1/3: > > - Modified verbiage in mma.md per Will's suggestion. > > - Modified rs6000_split_multireg_move to correctly handle BE PXImode > > and POImode moves. > > Patch 2/3: > > - Updated ChangeLog entry per Segher's suggestion. > > - Updated doc/extend.texi with correct built-in names for > > __builtin_vsx_xvcvspbf16 and __builtin_vsx_xvcvbf16sp. > > Patch 3/3: > > - No changes. > > The committed patches don't seem to have caused any bootstrap issues on trunk > and are pretty independent of the rest of the rs6000 backend, so I'd like > permission to back port the two commits to GCC 10. Patch 2 makes used of > the u8bit_cint_operand predicate which Kelvin added, but that isn't in GCC 10, > so I need to back port that change too. > > The back ports of the MMA patches/commits was straight forward and I'm > currently bootstrapping/regtesting the backports on both powerpc64le-linux > and powerpc64-linux. Is this (including the hunk below) ok for GCC 10 > release branch assuming the tests come back clean?
Yes, all are okay for 10 as well (incl. Kelvin's backport). Thanks! > rs6000: Backport u8bit_cint_operand predicate. (No dot at the end of the subject please.) Segher