Re: [PATCH] Pattern recognizer for rotates

2013-05-16 Thread Jakub Jelinek
On Wed, May 15, 2013 at 03:24:37PM +0200, Richard Biener wrote: > Ok with ... > > + /* Pattern detected. */ > > + if (dump_enabled_p ()) > > +dump_printf_loc (MSG_OPTIMIZED_LOCATIONS, vect_location, > > +"vect_recog_rotate_pattern: detected: "); > > Please use MSG_NOTE here

Re: [PATCH] Pattern recognizer for rotates

2013-05-15 Thread Richard Biener
On Fri, 10 May 2013, Jakub Jelinek wrote: > Hi! > > This patch allows us to vectorize attached testcases again, rot1.c > and rot3.c e.g. with -O3 -mavx, rot2.c only with -O3 -mavx2. > > The problem with rot2.c is that the rotation count is there > vect_external_def, but when we insert some stmts

[PATCH] Pattern recognizer for rotates

2013-05-10 Thread Jakub Jelinek
Hi! This patch allows us to vectorize attached testcases again, rot1.c and rot3.c e.g. with -O3 -mavx, rot2.c only with -O3 -mavx2. The problem with rot2.c is that the rotation count is there vect_external_def, but when we insert some stmts into the pattern def seq (the (-j) & 31), we turn it int