Re: [PATCH i386 AVX2] Remove redundant expands.

2013-10-16 Thread Uros Bizjak
On Wed, Oct 16, 2013 at 7:01 PM, Richard Henderson wrote: > On 10/16/2013 09:47 AM, Uros Bizjak wrote: >> On Wed, Oct 16, 2013 at 6:06 PM, Kirill Yukhin >> wrote: >> >>> It seems that gang of AVX* patterns were copy and >>> pasted from SSE, however as far as they are NDD, >>> we may remove corre

Re: [PATCH i386 AVX2] Remove redundant expands.

2013-10-16 Thread Richard Henderson
On 10/16/2013 09:47 AM, Uros Bizjak wrote: > On Wed, Oct 16, 2013 at 6:06 PM, Kirill Yukhin > wrote: > >> It seems that gang of AVX* patterns were copy and >> pasted from SSE, however as far as they are NDD, >> we may remove corresponding expands which sort operands. > > OTOH, I have some secon

Re: [PATCH i386 AVX2] Remove redundant expands.

2013-10-16 Thread Uros Bizjak
On Wed, Oct 16, 2013 at 6:06 PM, Kirill Yukhin wrote: > It seems that gang of AVX* patterns were copy and > pasted from SSE, however as far as they are NDD, > we may remove corresponding expands which sort operands. OTOH, I have some second thoughts on removing AVX2 expanders. Please consider t

Re: [PATCH i386 AVX2] Remove redundant expands.

2013-10-16 Thread Uros Bizjak
On Wed, Oct 16, 2013 at 6:06 PM, Kirill Yukhin wrote: > It seems that gang of AVX* patterns were copy and > pasted from SSE, however as far as they are NDD, > we may remove corresponding expands which sort operands. > > ChangeLog: > * config/i386/sse.md (vec_widen_umult_even_v8si): Remove

[PATCH i386 AVX2] Remove redundant expands.

2013-10-16 Thread Kirill Yukhin
Hello, It seems that gang of AVX* patterns were copy and pasted from SSE, however as far as they are NDD, we may remove corresponding expands which sort operands. ChangeLog: * config/i386/sse.md (vec_widen_umult_even_v8si): Remove expand, make insn visible, remove redundant check.