Re: [PATCH i386 AVX512] [30/n] Add FMA patterns.

2014-09-09 Thread Uros Bizjak
On Tue, Sep 9, 2014 at 1:28 PM, Kirill Yukhin wrote: > Hello Uroš, > On 28 Aug 20:44, Uros Bizjak wrote: >> On Thu, Aug 28, 2014 at 3:48 PM, Kirill Yukhin >> wrote: >> Please document these changes as: >> >> [new_pattern_name]: Rename from [old_pattern_name] and use VF_AVXyy >> mode iterator. >>

Re: [PATCH i386 AVX512] [30/n] Add FMA patterns.

2014-09-09 Thread Kirill Yukhin
Hello Uroš, On 28 Aug 20:44, Uros Bizjak wrote: > On Thu, Aug 28, 2014 at 3:48 PM, Kirill Yukhin > wrote: > Please document these changes as: > > [new_pattern_name]: Rename from [old_pattern_name] and use VF_AVXyy > mode iterator. > > > -(define_insn "fma_fmsub_" > > +(define_insn > > "fma_fms

Re: [PATCH i386 AVX512] [30/n] Add FMA patterns.

2014-08-29 Thread Kirill Yukhin
Hello Uroš, On 28 Aug 20:44, Uros Bizjak wrote: > On Thu, Aug 28, 2014 at 3:48 PM, Kirill Yukhin > wrote: > > Is it ok for trunk? > I'd suggest to put noavx512 at the beginning, so: > > "noavx512_fma_fmsub_" This is not possible currently since subst attribute is expanded to `*' when mask not a

Re: [PATCH i386 AVX512] [30/n] Add FMA patterns.

2014-08-28 Thread Uros Bizjak
On Thu, Aug 28, 2014 at 3:48 PM, Kirill Yukhin wrote: > Hello, > This patch adds patterns to support FMA new insns. > > Bootstrapped. > AVX-512* tests on top of patch-set all pass > under simulator. > > Is it ok for trunk? > > gcc/ > * config/i386/sse.md > (define_mode_iterator VF_

[PATCH i386 AVX512] [30/n] Add FMA patterns.

2014-08-28 Thread Kirill Yukhin
Hello, This patch adds patterns to support FMA new insns. Bootstrapped. AVX-512* tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ * config/i386/sse.md (define_mode_iterator VF_AVX512VL): New. (define_mode_iterator FMAMODEM): Allow 128/256bit ev