[Patch] x86: Enable support for Intel UINTR extension

Hongyu Wang wwwhhhyyy333@gmail.com
Wed Oct 14 09:06:20 GMT 2020


Uros Bizjak <ubizjak@gmail.com> 于2020年10月14日周三 下午4:53写道:
>
> On Wed, Oct 14, 2020 at 10:42 AM Uros Bizjak <ubizjak@gmail.com> wrote:
> >
> > On Wed, Oct 14, 2020 at 10:34 AM Hongyu Wang <wwwhhhyyy333@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.

Also for x86gprintrin-3.s, without -muintr:

_clui:
.LFB128:
        .cfi_startproc
        pushq   %rbp
        .cfi_def_cfa_offset 16
        .cfi_offset 6, -16
        movq    %rsp, %rbp
        .cfi_def_cfa_register 6
        clui
        nop
        popq    %rbp
        .cfi_def_cfa 7, 8
        ret
        .cfi_endproc

-- 
Regards,

Hongyu, Wang


More information about the Gcc-patches mailing list