Re: [PATCH] Replace const_vector with match_operand in sse.md

2012-11-20 Thread Kirill Yukhin
Hi, > OK for mainline SVN, if bootstrapped and regression tested appropriately. Checked in: http://gcc.gnu.org/ml/gcc-cvs/2012-11/msg00603.html Thanks, K

Re: [PATCH] Replace const_vector with match_operand in sse.md

2012-11-13 Thread Uros Bizjak
On Tue, Nov 13, 2012 at 3:23 PM, Andrey Turetskiy wrote: >> BTW: Probably, pmulhrsw insn patterns can be merged, too, but this can >> be a follow-up patch. > > Please, have a look at patch which merge pmulhrsw patterns. > > Changelog: > > 2012-11-13 Andrey Turetskiy > >* config/i386/sse

Re: [PATCH] Replace const_vector with match_operand in sse.md

2012-11-13 Thread Andrey Turetskiy
> BTW: Probably, pmulhrsw insn patterns can be merged, too, but this can > be a follow-up patch. Please, have a look at patch which merge pmulhrsw patterns. Changelog: 2012-11-13 Andrey Turetskiy * config/i386/sse.md (*_pmulhrsw3): Merge *avx2_pmulhrswv16hi3 and *ssse3_pmulhrsw

Re: [PATCH] Replace const_vector with match_operand in sse.md

2012-11-05 Thread Kirill Yukhin
Hi, > Yes, the patch is OK for mainline SVN. Checked into trunk: http://gcc.gnu.org/ml/gcc-cvs/2012-11/msg00155.html Thanks, K

Re: [PATCH] Replace const_vector with match_operand in sse.md

2012-10-30 Thread Uros Bizjak
On Tue, Oct 30, 2012 at 6:53 PM, Andrey Turetskiy wrote: > Thanks for explanation, I understand it. > I fixed issue which you marked. Changelog is unchanged. > >>> I changed the patch according Uros' remarks. Please, have a look. >>> >>> Changelog: >>> >>> 2012-10-30 Andrey Turetskiy >>> >>>

Re: [PATCH] Replace const_vector with match_operand in sse.md

2012-10-30 Thread Andrey Turetskiy
Thanks for explanation, I understand it. I fixed issue which you marked. Changelog is unchanged. --- Best regards, Andrey Turetskiy On Tue, Oct 30, 2012 at 7:40 PM, Uros Bizjak wrote: > On Tue, Oct 30, 2012 at 3:47 PM, Andrey Turetskiy > wrote: >> I changed the patch according Uros' remarks. Pl

Re: [PATCH] Replace const_vector with match_operand in sse.md

2012-10-30 Thread Uros Bizjak
On Tue, Oct 30, 2012 at 3:47 PM, Andrey Turetskiy wrote: > I changed the patch according Uros' remarks. Please, have a look. > > Changelog: > > 2012-10-30 Andrey Turetskiy > >* config/i386/i386.c (bdesc_args): Rename CODE_FOR_avx2_umulhrswv16hi3 > to >CODE_FOR_avx2_pmulhrswv16h

Re: [PATCH] Replace const_vector with match_operand in sse.md

2012-10-30 Thread Andrey Turetskiy
I changed the patch according Uros' remarks. Please, have a look. Changelog: 2012-10-30 Andrey Turetskiy * config/i386/i386.c (bdesc_args): Rename CODE_FOR_avx2_umulhrswv16hi3 to CODE_FOR_avx2_pmulhrswv16hi3. * config/i386/predicates.md (const1_operand): Extend for vector

Re: [PATCH] Replace const_vector with match_operand in sse.md

2012-10-24 Thread Uros Bizjak
On Wed, Oct 24, 2012 at 3:01 PM, Andrey Turetskiy wrote: > On Tue, Oct 23, 2012 at 2:45 PM, Andrey Turetskiy > wrote: >> Hi, >> >> This patch replaces large const_vector constructions with >> match_operand in sse.md to decrease its size. >> Is it ok? No, you don't have to touch generic expand m

Re: [PATCH] Replace const_vector with match_operand in sse.md

2012-10-24 Thread Jakub Jelinek
On Wed, Oct 24, 2012 at 05:01:15PM +0400, Andrey Turetskiy wrote: > > This patch replaces large const_vector constructions with > > match_operand in sse.md to decrease its size. > > Is it ok? The *intrin.h changes look all wrong to me, why should one pass a dummy uninitialized argument to the buil

Re: [PATCH] Replace const_vector with match_operand in sse.md

2012-10-24 Thread Andrey Turetskiy
Add Uros to Cc. On Tue, Oct 23, 2012 at 2:45 PM, Andrey Turetskiy wrote: > Hi, > > This patch replaces large const_vector constructions with > match_operand in sse.md to decrease its size. > Is it ok? > > Changelog: > > 2012-10-23 Andrey Turetskiy > >* config/i386/avx2intrin.h (_mm256_avg_