Re: Repost [PATCH 5/6] PowerPC: Switch to dense math names for all MMA operations.

[Michael Meissner <meissner@linux.ibm.com>]

On Sun, Feb 04, 2024 at 01:47:12PM +0800, Kewen.Lin wrote:
> > diff --git a/gcc/config/rs6000/mma.md b/gcc/config/rs6000/mma.md
> > index 525a85146ff..f06e6bbb184 100644
> > --- a/gcc/config/rs6000/mma.md
> > +++ b/gcc/config/rs6000/mma.md
> > @@ -227,13 +227,22 @@ (define_int_attr apv		[(UNSPEC_MMA_XVF64GERPP		"xvf64gerpp")
> >  
> >  (define_int_attr vvi4i4i8	[(UNSPEC_MMA_PMXVI4GER8		"pmxvi4ger8")])
> >  
> > +(define_int_attr vvi4i4i8_dm	[(UNSPEC_MMA_PMXVI4GER8		"pmdmxvi4ger8")])
> 
> Can we update vvi4i4i8 to
> 
> (define_int_attr vvi4i4i8	[(UNSPEC_MMA_PMXVI4GER8		"xvi4ger8")])
> 
> by avoiding to introduce vvi4i4i8_dm, then its use places would be like:
> 
> -  "<vvi4i4i8> %A0,%x1,%x2,%3,%4,%5"
> +  "@
> +   pmdm<vvi4i4i8> %A0,%x1,%x2,%3,%4,%5
> +   pm<vvi4i4i8> %A0,%x1,%x2,%3,%4,%5
> +   pm<vvi4i4i8> %A0,%x1,%x2,%3,%4,%5"
> 
> and 
> 
> - define_insn "mma_<vvi4i4i8>"
> + define_insn "mma_pm<vvi4i4i8>"
> 
> (or updating its use in corresponding bif expander field)

Yes I can do that.

-- 
Michael Meissner, IBM
PO Box 98, Ayer, Massachusetts, USA, 01432
email: meissner@linux.ibm.com