[PATCH] Enable GCC support for AVX512_VP2INTERSECT.

[Hongtao Liu]

On Fri, Jun 21, 2019 at 1:56 PM Uros Bizjak <ubizjak@gmail.com> wrote:
>
> On Fri, Jun 21, 2019 at 4:21 AM Hongtao Liu <crazylht@gmail.com> wrote:
> >
> > On Thu, Jun 20, 2019 at 10:58 PM H.J. Lu <hjl.tools@gmail.com> wrote:
> > >
> > > On Thu, Jun 20, 2019 at 3:54 AM Hongtao Liu <crazylht@gmail.com> wrote:
> > > >
> > > > On Thu, Jun 20, 2019 at 2:13 PM Uros Bizjak <ubizjak@gmail.com> wrote:
> > > > >
> > > > > On Thu, Jun 20, 2019 at 7:36 AM Hongtao Liu <crazylht@gmail.com> wrote:
> > > > > >
> > > > > > On Sat, Jun 8, 2019 at 4:12 AM Uros Bizjak <ubizjak@gmail.com> wrote:
> > > > > > >
> > > > > > > On 6/7/19, H.J. Lu <hjl.tools@gmail.com> wrote:
> > > > > > >
> > > > > > > >> > > +/* Register pair.  */
> > > > > > > >> > > +VECTOR_MODES_WITH_PREFIX (P, INT, 2); /* P2QI */
> > > > > > > >> > > +VECTOR_MODES_WITH_PREFIX (P, INT, 4); /* P2HI P4QI */
> > > > > > > >> > >
> > > > > > > >> > > I think
> > > > > > > >> > >
> > > > > > > >> > > INT_MODE (P2QI, 16);
> > > > > > > >> > > INT_MODE (P2HI, 32);
> > Why P2QI need 16 bytes but not 2 bytes?
> > Same question with P2HI.
>
> Because we made a mistake. It should be 2 and 4, since these arguments
Then it will run into internal comiler error when building libgcc.
I'm still invertigating it.
> are bytes, not bits.
>
> This will also fix gen_highpart issue.
>
> Uros.
>
> > > > > > > >> > >
> > > > > > > >> > > with the above subreg approach should work.
> > > > Yes, it works.
> > > >
> > > > But i didn't figure out how did pass_reload correctly handle such subreg,
> > > > do you have suggestions such as "which function i can dig into first" or
> > > > "which piece of codes handle subreg"?
> > >
> > > You need to define REGMODE_NATURAL_SIZE.
> > >
> > > --
> > > H.J.
> >
> >
> >
> > --
> > BR,
> > Hongtao



-- 
BR,
Hongtao