[PATCH i386 AVX512] [63.1/n] Add vpshufb, perm autogen (except for v64qi).

Jakub Jelinek jakub@redhat.com
Thu Oct 16 11:18:00 GMT 2014


On Thu, Oct 16, 2014 at 02:23:16PM +0400, Ilya Tocar wrote:
> On 10 Oct 18:37, Uros Bizjak wrote:
> > On Fri, Oct 10, 2014 at 5:47 PM, Ilya Tocar <tocarip.intel@gmail.com> wrote:
> > 
> > 
> > Please recode that horrible first switch statement to:
> > 
> > --cut here--
> >   rtx (*gen) (rtx, rtx, rtx, rtx) = NULL;
> > 
> >   switch (mode)
> >     {
> >     case V8HImode:
> >       if (TARGET_AVX512VL && TARGET_AVX152BW)
> >     gen = gen_avx512vl_vpermi2varv8hi3;
> >       break;
> > 
> >     ...
> > 
> >     case V2DFmode:
> >       if (TARGET_AVX512VL)
> >     {
> >       gen = gen_avx512vl_vpermi2varv2df3;
> >       maskmode = V2DImode;
> > 
> > The patch is OK with the above improvement.
> > 
> > Thanks,
> > Uros.
> >
> 
> Will commit version below, if no objections in 24 hours.

No need to wait, it is ok now (with proper ChangeLog of course).

	Jakub



More information about the Gcc-patches mailing list