On Wed, Oct 14, 2020 at 10:42 AM Uros Bizjak <ubiz...@gmail.com> wrote:
>
> On Wed, Oct 14, 2020 at 10:34 AM Hongyu Wang <wwwhhhyyy...@gmail.com> wrote:
> >
> > >
> > > Please also add -muintr to g++.dg/other/i386-{2,3}.C and
> > > gcc.target/i386-sse-{12,13,14,22,23}.c. This will test new intrinsics
> > > header.
> > >
> >
> > Thanks for your review. We found that without adding -muintr, the
> > intrinsics header could also be tested. Make-check for these file all get
> > passed.
> >
> > And there is no intrinsic/builtin with const int parameter. So we remove
> > -muintr from these files.
>
> Can your double check that relevant instructions are indeed generated?
> Without -muintr, relevant patterns in i386.md are effectively blocked,
> and perhaps a call to __builtin_ia32_* is generated instead.
Ah, I see the issue.
The new header should be tested via x86gprintrin-* test cases.
Uros.
