Re: Dump before flag

2011-06-14 Thread Xinliang David Li
Here is one the of follow up patches: support of -before_preparation, -before, -after, -after_cleanup dump flags. The default dumping behavior does not change at all, but if any one of the above flags is specified, the function IR will be dumped into a file with the corresponding suffix. The enh

Re: Dump before flag

2011-06-14 Thread Xinliang David Li
Committed after Bootstrapping and regression testing on x86-64/linux. The follow up patch will come soon. Thanks, David On Tue, Jun 14, 2011 at 8:57 AM, Xinliang David Li wrote: > On Tue, Jun 14, 2011 at 6:58 AM, Richard Guenther > wrote: >> On Fri, Jun 10, 2011 at 8:44 PM, Xinliang David Li

Re: Dump before flag

2011-06-14 Thread Xinliang David Li
On Tue, Jun 14, 2011 at 6:58 AM, Richard Guenther wrote: > On Fri, Jun 10, 2011 at 8:44 PM, Xinliang David Li wrote: >> This is the revised patch as suggested. >> >> How does it look? > >  } > > +static void > +execute_function_dump (void *data ATTRIBUTE_UNUSED) > > function needs a comment. > >

Re: Dump before flag

2011-06-14 Thread Richard Guenther
On Fri, Jun 10, 2011 at 8:44 PM, Xinliang David Li wrote: > This is the revised patch as suggested. > > How does it look? } +static void +execute_function_dump (void *data ATTRIBUTE_UNUSED) function needs a comment. Ok with that change. Please always specify how you tested the patch - the pa

Re: Dump before flag

2011-06-10 Thread Xinliang David Li
On Fri, Jun 10, 2011 at 1:52 AM, Richard Guenther wrote: > On Thu, Jun 9, 2011 at 5:47 PM, Xinliang David Li wrote: >> See attached. > > Hmm.  I don't like how you still wire dumping in the TODO routines. > Doesn't it work to just dump the body from pass_fini_dump_file ()? > Or if that doesn't so

Re: Dump before flag

2011-06-10 Thread Richard Guenther
On Thu, Jun 9, 2011 at 5:47 PM, Xinliang David Li wrote: > See attached. Hmm. I don't like how you still wire dumping in the TODO routines. Doesn't it work to just dump the body from pass_fini_dump_file ()? Or if that doesn't sound clean from (a subset of) places where it is called? (we might wa

Re: Dump before flag

2011-06-09 Thread Richard Guenther
On Thu, Jun 9, 2011 at 12:31 AM, Xinliang David Li wrote: > this is the patch that just removes the TODO_dump flag and forces it > to dump. The original code cfun->last_verified = flags & > TODO_verify_all looks weird -- depending on TODO_dump is set or not, > the behavior of the update is differe

Re: Dump before flag

2011-06-08 Thread Xinliang David Li
this is the patch that just removes the TODO_dump flag and forces it to dump. The original code cfun->last_verified = flags & TODO_verify_all looks weird -- depending on TODO_dump is set or not, the behavior of the update is different (when no other todo flags is set). Ok for trunk? David On Wed

Re: Dump before flag

2011-06-08 Thread Xinliang David Li
On Wed, Jun 8, 2011 at 2:06 AM, Richard Guenther wrote: > On Wed, Jun 8, 2011 at 1:08 AM, Xinliang David Li wrote: >> The following is the patch that does the job. Most of the changes are >> just  removing TODO_dump_func. The major change is in passes.c and >> tree-pass.h. >> >> -fdump-xxx-yyy-st

Re: Dump before flag

2011-06-08 Thread Xinliang David Li
On Wed, Jun 8, 2011 at 2:06 AM, Richard Guenther wrote: > On Wed, Jun 8, 2011 at 1:08 AM, Xinliang David Li wrote: >> The following is the patch that does the job. Most of the changes are >> just  removing TODO_dump_func. The major change is in passes.c and >> tree-pass.h. >> >> -fdump-xxx-yyy-st

Re: Dump before flag

2011-06-08 Thread Richard Guenther
On Wed, Jun 8, 2011 at 1:08 AM, Xinliang David Li wrote: > The following is the patch that does the job. Most of the changes are > just  removing TODO_dump_func. The major change is in passes.c and > tree-pass.h. > > -fdump-xxx-yyy-start       <-- dump before TODO_start > -fdump-xxx-yyy-before    

Re: Dump before flag

2011-06-07 Thread Xinliang David Li
It might be also useful to implement the dumping behavior like this: if any of the start/before/after/finish option is explicitly specified, IR (and only IR) will be dumped into files suffixed with .start/.before/.after/.finish. The debug dump will be dumped as usual into the non suffixed file name

Re: Dump before flag

2011-06-07 Thread Diego Novillo
On Tue, Jun 7, 2011 at 10:06, Xinliang David Li wrote: > On Tue, Jun 7, 2011 at 10:01 AM, Diego Novillo wrote: >> On Tue, Jun 7, 2011 at 09:51, Xinliang David Li wrote: >>> Any suggestions on the dump position specification string, before and >>> after is not enough. How about >>> >>> start, bef

Re: Dump before flag

2011-06-07 Thread Xinliang David Li
On Tue, Jun 7, 2011 at 10:01 AM, Diego Novillo wrote: > On Tue, Jun 7, 2011 at 09:51, Xinliang David Li wrote: >> Any suggestions on the dump position specification string, before and >> after is not enough. How about >> >> start, before, after, and finish? >> >> I.e. >> >> -fdump-tree-pre-start

Re: Dump before flag

2011-06-07 Thread Diego Novillo
On Tue, Jun 7, 2011 at 09:51, Xinliang David Li wrote: > Any suggestions on the dump position specification string, before and > after is not enough. How about > > start, before, after, and finish? > > I.e. > > -fdump-tree-pre-start --> dump IR before TODO_start of PRE pass > -fdump-tree-pre-befor

Re: Dump before flag

2011-06-07 Thread Xinliang David Li
Any suggestions on the dump position specification string, before and after is not enough. How about start, before, after, and finish? I.e. -fdump-tree-pre-start --> dump IR before TODO_start of PRE pass -fdump-tree-pre-before --> dump IR just before PRE after its TODO start finishes -fdump-tree

Re: Dump before flag

2011-06-07 Thread Diego Novillo
On Tue, Jun 7, 2011 at 02:36, Richard Guenther wrote: >> For one thing, you need to either remember what is the previous pass, >> or dump all passes which for large files can take very long time. Even >> with all the dumps, you will need to eyeballing to find the previous >> pass which may or may

Re: Dump before flag

2011-06-07 Thread Xinliang David Li
On Tue, Jun 7, 2011 at 2:36 AM, Richard Guenther wrote: > On Mon, Jun 6, 2011 at 6:20 PM, Xinliang David Li wrote: >>> >>> Your patch doesn't really improve this but adds to the confusion. >>> >>> +  /* Override dump TODOs.  */ >>> +  if (dump_file && (pass->todo_flags_finish & TODO_dump_func) >>

Re: Dump before flag

2011-06-07 Thread Richard Guenther
On Mon, Jun 6, 2011 at 6:20 PM, Xinliang David Li wrote: >> >> Your patch doesn't really improve this but adds to the confusion. >> >> +  /* Override dump TODOs.  */ >> +  if (dump_file && (pass->todo_flags_finish & TODO_dump_func) >> +      && (dump_flags & TDF_BEFORE)) >> +    { >> +      pass->

Re: Dump before flag

2011-06-06 Thread Xinliang David Li
> > Your patch doesn't really improve this but adds to the confusion. > > +  /* Override dump TODOs.  */ > +  if (dump_file && (pass->todo_flags_finish & TODO_dump_func) > +      && (dump_flags & TDF_BEFORE)) > +    { > +      pass->todo_flags_finish &= ~TODO_dump_func; > +      pass->todo_flags_st

Re: Dump before flag

2011-06-06 Thread Richard Guenther
On Wed, Jun 1, 2011 at 11:23 PM, Xinliang David Li wrote: > On Wed, Jun 1, 2011 at 2:12 PM, Basile Starynkevitch > wrote: >> On Wed, 1 Jun 2011 13:26:24 -0700 >> Xinliang David Li wrote: >> >>> Hi, this is a simple patch that support dump_before flag. E.g, >>> >>> -fdump-tree-pre-before >>> >>>

Re: Dump before flag

2011-06-01 Thread Xinliang David Li
On Wed, Jun 1, 2011 at 2:12 PM, Basile Starynkevitch wrote: > On Wed, 1 Jun 2011 13:26:24 -0700 > Xinliang David Li wrote: > >> Hi, this is a simple patch that support dump_before flag. E.g, >> >> -fdump-tree-pre-before >> >> This is useful for diffing the the IR before and after a pass. > > Perh

Re: Dump before flag

2011-06-01 Thread Basile Starynkevitch
On Wed, 1 Jun 2011 13:26:24 -0700 Xinliang David Li wrote: > Hi, this is a simple patch that support dump_before flag. E.g, > > -fdump-tree-pre-before > > This is useful for diffing the the IR before and after a pass. Perhaps you forgot to actually attach the patch? > Gcc dumping needs more c

Re: Dump before flag

2011-06-01 Thread Xinliang David Li
Sorry about it. Here it is. David On Wed, Jun 1, 2011 at 1:36 PM, Richard Guenther wrote: > On Wed, Jun 1, 2011 at 10:26 PM, Xinliang David Li wrote: >> Hi, this is a simple patch that support dump_before flag. E.g, >> >> -fdump-tree-pre-before >> >> This is useful for diffing the the IR befor

Re: Dump before flag

2011-06-01 Thread Richard Guenther
On Wed, Jun 1, 2011 at 10:26 PM, Xinliang David Li wrote: > Hi, this is a simple patch that support dump_before flag. E.g, > > -fdump-tree-pre-before > > This is useful for diffing the the IR before and after a pass. > > Gcc dumping needs more cleanups -- such as allowing IR only dump, > allowing

Dump before flag

2011-06-01 Thread Xinliang David Li
Hi, this is a simple patch that support dump_before flag. E.g, -fdump-tree-pre-before This is useful for diffing the the IR before and after a pass. Gcc dumping needs more cleanups -- such as allowing IR only dump, allowing IR dumping for a particular function etc. The exposure of 'dumpfile' (in