Re: [PATCH] Make vectorizer dumps more comparable

2011-09-01 Thread Richard Sandiford
Michael Matz writes: >> Ira Rosen writes: >> >> How about, as Micha suggested, print the location of the loop >> >> we currently investigate from vectorize_loops () where we >> >> call find_loop_location () instead? >> > >> > The problem is that a dump of a single loop can be pretty long, and "st

Re: [PATCH] Make vectorizer dumps more comparable

2011-09-01 Thread Michael Matz
Hi, On Thu, 1 Sep 2011, Richard Sandiford wrote: > Ira Rosen writes: > >> How about, as Micha suggested, print the location of the loop > >> we currently investigate from vectorize_loops () where we > >> call find_loop_location () instead? > > > > The problem is that a dump of a single loop can

Re: [PATCH] Make vectorizer dumps more comparable

2011-09-01 Thread Ira Rosen
Richard Guenther wrote on 01/09/2011 12:26:25 PM: > > > Well, it seems to be different what everybody else does and it's > > > highly redundant for a whole bunch of lines. > > > > > > But, it solves my diff issue and the overly long lines as well. > > > > > > Your patch changes both dump-file a

Re: [PATCH] Make vectorizer dumps more comparable

2011-09-01 Thread Richard Guenther
On Thu, 1 Sep 2011, Ira Rosen wrote: > > > Richard Guenther wrote on 01/09/2011 11:13:29 AM: > > > > > > IMO it's a bad idea. It's now impossible to find anything when > > > compiling a > > > > > big file. How about only removing the file name? > > > > > > > > How about, as Micha suggested, pr

Re: [PATCH] Make vectorizer dumps more comparable

2011-09-01 Thread Ira Rosen
Richard Guenther wrote on 01/09/2011 11:13:29 AM: > > > > IMO it's a bad idea. It's now impossible to find anything when > > compiling a > > > > big file. How about only removing the file name? > > > > > > How about, as Micha suggested, print the location of the loop > > > we currently investig

Re: [PATCH] Make vectorizer dumps more comparable

2011-09-01 Thread Richard Guenther
On Thu, 1 Sep 2011, Ira Rosen wrote: > > > Richard Guenther wrote on 01/09/2011 10:33:23 AM: > > > On Thu, 1 Sep 2011, Ira Rosen wrote: > > > > > > > > > > > gcc-patches-ow...@gcc.gnu.org wrote on 24/08/2011 02:20:50 PM: > > > > > > > > This avoids the file/location clutter in front of each li

Re: [PATCH] Make vectorizer dumps more comparable

2011-09-01 Thread Richard Sandiford
Ira Rosen writes: >> How about, as Micha suggested, print the location of the loop >> we currently investigate from vectorize_loops () where we >> call find_loop_location () instead? > > The problem is that a dump of a single loop can be pretty long, and "start > to analyze loop..."/"finish to ana

Re: [PATCH] Make vectorizer dumps more comparable

2011-09-01 Thread Ira Rosen
Richard Guenther wrote on 01/09/2011 10:33:23 AM: > On Thu, 1 Sep 2011, Ira Rosen wrote: > > > > > > > gcc-patches-ow...@gcc.gnu.org wrote on 24/08/2011 02:20:50 PM: > > > > > > This avoids the file/location clutter in front of each line > > > in the vectorizer dump. While this is useful for p

Re: [PATCH] Make vectorizer dumps more comparable

2011-09-01 Thread Richard Guenther
On Thu, 1 Sep 2011, Ira Rosen wrote: > > > gcc-patches-ow...@gcc.gnu.org wrote on 24/08/2011 02:20:50 PM: > > > > This avoids the file/location clutter in front of each line > > in the vectorizer dump. While this is useful for people > > requesting -fvectorizer-verbose=N in dump files this make

Re: [PATCH] Make vectorizer dumps more comparable

2011-09-01 Thread Ira Rosen
gcc-patches-ow...@gcc.gnu.org wrote on 24/08/2011 02:20:50 PM: > > This avoids the file/location clutter in front of each line > in the vectorizer dump. While this is useful for people > requesting -fvectorizer-verbose=N in dump files this makes > you unable to compare dumps for testcases on a b

Re: [PATCH] Make vectorizer dumps more comparable

2011-08-24 Thread Richard Guenther
On Wed, 24 Aug 2011, Michael Matz wrote: > Hi, > > On Wed, 24 Aug 2011, Richard Guenther wrote: > > > This avoids the file/location clutter in front of each line in the > > vectorizer dump. While this is useful for people requesting > > -fvectorizer-verbose=N in dump files this makes you unab

Re: [PATCH] Make vectorizer dumps more comparable

2011-08-24 Thread Michael Matz
Hi, On Wed, 24 Aug 2011, Richard Guenther wrote: > This avoids the file/location clutter in front of each line in the > vectorizer dump. While this is useful for people requesting > -fvectorizer-verbose=N in dump files this makes you unable to compare > dumps for testcases on a branch and tru

[PATCH] Make vectorizer dumps more comparable

2011-08-24 Thread Richard Guenther
This avoids the file/location clutter in front of each line in the vectorizer dump. While this is useful for people requesting -fvectorizer-verbose=N in dump files this makes you unable to compare dumps for testcases on a branch and trunk. It also makes lines excessively long because the testsuit