Re: [PATCH i386 AVX512] [20/n] AVX-512 integer shift pattern.

2014-08-20 Thread Uros Bizjak
On Wed, Aug 20, 2014 at 3:08 PM, Kirill Yukhin wrote: > Hello, > On 15 Aug 20:35, Uros Bizjak wrote: >> On Fri, Aug 15, 2014 at 1:56 PM, Kirill Yukhin >> wrote: >> Again, please split insn pattern to avoid: >> >> + "TARGET_SSE2 >> + && >> + && ((mode != V16HImode && mode != V8HImode) >> +

Re: [PATCH i386 AVX512] [20/n] AVX-512 integer shift pattern.

2014-08-20 Thread Kirill Yukhin
Hello, On 15 Aug 20:35, Uros Bizjak wrote: > On Fri, Aug 15, 2014 at 1:56 PM, Kirill Yukhin > wrote: > Again, please split insn pattern to avoid: > > + "TARGET_SSE2 > + && > + && ((mode != V16HImode && mode != V8HImode) > + || TARGET_AVX512BW > + || !)" > > insn constraints. T

Re: [PATCH i386 AVX512] [20/n] AVX-512 integer shift pattern.

2014-08-15 Thread Uros Bizjak
On Fri, Aug 15, 2014 at 1:56 PM, Kirill Yukhin wrote: > Hello, > This patch extends shift pattern to support AVX-512 > new insn. > > Bootstrapped. > New tests on top of patch-set all pass > under simulator. > > Is it ok for trunk? > > gcc/ > * config/i386/sse.md > (define_mode_iter

[PATCH i386 AVX512] [20/n] AVX-512 integer shift pattern.

2014-08-15 Thread Kirill Yukhin
Hello, This patch extends shift pattern to support AVX-512 new insn. Bootstrapped. New tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ * config/i386/sse.md (define_mode_iterator VI248_AVX2): Add V32HI mode. (define_insn "3"): Add masking. --