Re: [PATCH i386 AVX512] [19/n] Extends AVX-512 broadcasts.

2014-08-20 Thread Uros Bizjak
On Wed, Aug 20, 2014 at 2:51 PM, Kirill Yukhin wrote: > Hello Uroš, > On 15 Aug 20:29, Uros Bizjak wrote: >> Can you avoid insn constraints like: >> >> > + "TARGET_AVX512DQ && ( == 64 || TARGET_AVX512VL)" >> >> This should be split to two insn patterns, each with different >> baseline insn constr

Re: [PATCH i386 AVX512] [19/n] Extends AVX-512 broadcasts.

2014-08-20 Thread Kirill Yukhin
Hello Uroš, On 15 Aug 20:29, Uros Bizjak wrote: > Can you avoid insn constraints like: > > > + "TARGET_AVX512DQ && ( == 64 || TARGET_AVX512VL)" > > This should be split to two insn patterns, each with different > baseline insn constraint. I've splitted pattern into two similar w/ different mode

Re: [PATCH i386 AVX512] [19/n] Extends AVX-512 broadcasts.

2014-08-15 Thread Uros Bizjak
On Fri, Aug 15, 2014 at 1:52 PM, Kirill Yukhin wrote: > Hello, > This patch introduces new patterns to support > AVX-512Vl,DQ broadcast insns. > > Bootstrapped. > New tests on top of patch-set all pass > under simulator. > > Is it ok for trunk? > > gcc/ > * config/i386/sse.md > (de

[PATCH i386 AVX512] [19/n] Extends AVX-512 broadcasts.

2014-08-15 Thread Kirill Yukhin
Hello, This patch introduces new patterns to support AVX-512Vl,DQ broadcast insns. 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 VI4F_BRCST32x2): New. (define_mode_attr 64x2_mode):