Uros Bizjak <ubiz...@gmail.com> 于2020年10月14日周三 下午4:53写道: > > 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.
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