Re: [PATCH] Intrinsics for PREFETCHW

2012-07-30 Thread Uros Bizjak
On Mon, Jul 30, 2012 at 2:05 PM, Kirill Yukhin wrote: >> Ehm ... >> >>> * gcc.target/i386/sse-13.c: Ditto. >>> * gcc.target/i386/sse-14.c: Ditto. >>> * g++.dg/other/i386-2.C: Ditto. >>> * g++.dg/other/i386-3.C: Ditto. > Sorry, what's wrong here? Not here, but abov

Re: [PATCH] Intrinsics for PREFETCHW

2012-07-30 Thread Kirill Yukhin
> Ehm ... > >> * gcc.target/i386/sse-13.c: Ditto. >> * gcc.target/i386/sse-14.c: Ditto. >> * g++.dg/other/i386-2.C: Ditto. >> * g++.dg/other/i386-3.C: Ditto. Sorry, what's wrong here? > I suggest you implement handling of this builtin in the same way > rdrand_1 is i

Re: [PATCH] Intrinsics for PREFETCHW

2012-07-25 Thread Uros Bizjak
On Wed, Jul 25, 2012 at 4:07 PM, Kirill Yukhin wrote: > Here is second patch which adds support of rdseed[16|32|64] insn. > > Changelog entry: > 2012-07-25 Kirill Yukhin > Michael Zolotukhin > > * common/config/i386/i386-common.c (OPTION_MASK_ISA_RDSEED_SET): New. >

Re: [PATCH] Intrinsics for PREFETCHW

2012-07-25 Thread Kirill Yukhin
Hi again, Here is second patch which adds support of rdseed[16|32|64] insn. Changelog entry: 2012-07-25 Kirill Yukhin Michael Zolotukhin * common/config/i386/i386-common.c (OPTION_MASK_ISA_RDSEED_SET): New. (OPTION_MASK_ISA_RDSEED_UNSET): Likewise. (ix86_h

Re: [PATCH] Intrinsics for PREFETCHW

2012-07-25 Thread Kirill Yukhin
>> Is it OK for trunk? > > OK. Thanks! Checked in. http://gcc.gnu.org/viewcvs?view=revision&revision=189844 Next I think would be rdseed* insns. Thanks, K

Re: [PATCH] Intrinsics for PREFETCHW

2012-07-25 Thread Uros Bizjak
On Wed, Jul 25, 2012 at 1:51 PM, Kirill Yukhin wrote: >> Please see attached patch that does all this with minimum surgery and >> also sets missing SSE prefetch for x86_32 with -mprfchw > Thanks, applied! > >> +++ b/gcc/config/i386/prfchwintrin.h >> >> +#if !defined _X86INTRIN_H_INCLUDED && !defin

Re: [PATCH] Intrinsics for PREFETCHW

2012-07-25 Thread Kirill Yukhin
> Please see attached patch that does all this with minimum surgery and > also sets missing SSE prefetch for x86_32 with -mprfchw Thanks, applied! > +++ b/gcc/config/i386/prfchwintrin.h > > +#if !defined _X86INTRIN_H_INCLUDED && !defined _MM3DNOW_H_INCLUDED > +# error "Never use directly; include

Re: [PATCH] Intrinsics for PREFETCHW

2012-07-24 Thread Uros Bizjak
On Tue, Jul 24, 2012 at 4:06 PM, Kirill Yukhin wrote: >> Can you please split the patch to three independent parts, prefetchw >> handling, rdseed and adxx. I got lost in the patch the second time. > Sure! > >> I propose we start with prefetch. You are still changing prefetch >> patterns, while it