Re: [PATCH i386 AVX512] [8/n] Extend substs for new patterns.

2014-08-13 Thread Uros Bizjak
On Wed, Aug 13, 2014 at 4:23 PM, Kirill Yukhin wrote: >> >> In a couple of places, checks are changed to GET_MODE_SIZE >> >> (GET_MODE (operands[0])) and in a similar way, mode checks were >> >> changed to GET_MODE (operands[0]). The previous checks are more >> >> efficient and are preferred. Is

Re: [PATCH i386 AVX512] [8/n] Extend substs for new patterns.

2014-08-13 Thread Kirill Yukhin
On 13 Aug 15:21, Uros Bizjak wrote: > On Wed, Aug 13, 2014 at 2:14 PM, Kirill Yukhin > wrote: > > >> In a couple of places, checks are changed to GET_MODE_SIZE > >> (GET_MODE (operands[0])) and in a similar way, mode checks were > >> changed to GET_MODE (operands[0]). The previous checks are mo

Re: [PATCH i386 AVX512] [8/n] Extend substs for new patterns.

2014-08-13 Thread Uros Bizjak
On Wed, Aug 13, 2014 at 2:14 PM, Kirill Yukhin wrote: >> In a couple of places, checks are changed to GET_MODE_SIZE >> (GET_MODE (operands[0])) and in a similar way, mode checks were >> changed to GET_MODE (operands[0]). The previous checks are more >> efficient and are preferred. Is there a rea

Re: [PATCH i386 AVX512] [8/n] Extend substs for new patterns.

2014-08-13 Thread Kirill Yukhin
Hello Uroš, On 12 Aug 20:16, Uros Bizjak wrote: > In a couple of places, checks are changed to GET_MODE_SIZE > (GET_MODE (operands[0])) and in a similar way, mode checks were > changed to GET_MODE (operands[0]). The previous checks are more > efficient and are preferred. Is there a reason you need

Re: [PATCH i386 AVX512] [8/n] Extend substs for new patterns.

2014-08-12 Thread Uros Bizjak
On Tue, Aug 12, 2014 at 2:58 PM, Kirill Yukhin wrote: > Hello, > This patch extends substs/subst_attrs to be used > with new patterns. > > Bootstrapped. > > Is it ok for trunk? In a couple of places, checks are changed to GET_MODE_SIZE (GET_MODE (operands[0])) and in a similar way, mode checks w

[PATCH i386 AVX512] [8/n] Extend substs for new patterns.

2014-08-12 Thread Kirill Yukhin
Hello, This patch extends substs/subst_attrs to be used with new patterns. Bootstrapped. Is it ok for trunk? gcc/ * config/i386/sse.md: Allow V64QI, V32QI, V32HI, V4HI modes. * config/i386/subst.md (define_mode_iterator SUBST_V): Update. (define_mode_iterator SUBS