Re: [patch] cluster loop bodies in CFG graph dumping

2012-12-12 Thread Richard Biener
On Wed, Dec 12, 2012 at 1:08 PM, Steven Bosscher wrote: > On Wed, Dec 12, 2012 at 11:46 AM, Richard Biener wrote: >> On Tue, Dec 11, 2012 at 8:57 PM, Steven Bosscher wrote: >> I'm not sure we want to give up this: >> >> @@ -863,12 +863,13 @@ get_loop_body_in_dom_order (const struct loop *loo >>

Re: [patch] cluster loop bodies in CFG graph dumping

2012-12-12 Thread Steven Bosscher
On Wed, Dec 12, 2012 at 11:46 AM, Richard Biener wrote: > On Tue, Dec 11, 2012 at 8:57 PM, Steven Bosscher wrote: > I'm not sure we want to give up this: > > @@ -863,12 +863,13 @@ get_loop_body_in_dom_order (const struct loop *loo >basic_block *tovisit; >int tv; > > + if (loop->latch == EX

Re: [patch] cluster loop bodies in CFG graph dumping

2012-12-12 Thread Richard Biener
On Tue, Dec 11, 2012 at 8:57 PM, Steven Bosscher wrote: > Hello, > > This patch clusters loop bodies in the CFG graph dumper. Makes for > easier-to-interpret plots and helps debugging. > > Bootstrapped&tested on powerpc64-unknown-linux-gnu. OK for trunk? I'm not sure we want to give up this: @@

[patch] cluster loop bodies in CFG graph dumping

2012-12-11 Thread Steven Bosscher
Hello, This patch clusters loop bodies in the CFG graph dumper. Makes for easier-to-interpret plots and helps debugging. Bootstrapped&tested on powerpc64-unknown-linux-gnu. OK for trunk? Ciao! Steven graph_dump_loops.diff Description: Binary data