Ping: [PATCH] Add -fdump-rtl--quiet

2012-05-09 Thread Andrew Stubbs
Ping. On 02/05/12 10:49, Andrew Stubbs wrote: On 19/04/12 13:58, Andrew Stubbs wrote: In the meantime, Mr Maintainers, can I commit my patch while we wait for the new world order? I'm happy to change the option name "quiet" to something else if necessary. Ping. I think David may have a point

Re: [PATCH] Add -fdump-rtl--quiet

2012-05-02 Thread Andrew Stubbs
On 19/04/12 13:58, Andrew Stubbs wrote: In the meantime, Mr Maintainers, can I commit my patch while we wait for the new world order? I'm happy to change the option name "quiet" to something else if necessary. Ping. I think David may have a point about 'quiet' being an inappropriate name for

Re: [PATCH] Add -fdump-rtl--quiet

2012-04-19 Thread Andrew Stubbs
On 19/04/12 17:21, Xinliang David Li wrote: To me, the existing "(default), verbose, details" option set makes sense, and adding "quiet" to that set seemed appropriate. I'm not sure I like the idea of renaming existing options, even if they aren't likely to break much. It is confusing. 'verbose

Re: [PATCH] Add -fdump-rtl--quiet

2012-04-19 Thread Xinliang David Li
On Thu, Apr 19, 2012 at 5:58 AM, Andrew Stubbs wrote: > On 18/04/12 22:09, Xinliang David Li wrote: >> >> Flags in category 1) >> -- >> There are four types of information that can be dumped (should be >> controlled by flag set 1) ): >>   1.a) transformation decisions -- su

Re: [PATCH] Add -fdump-rtl--quiet

2012-04-19 Thread Andrew Stubbs
On 18/04/12 22:09, Xinliang David Li wrote: Flags in category 1) -- There are four types of information that can be dumped (should be controlled by flag set 1) ): 1.a) transformation decisions -- such as -fdump-tree-vectorize-transformations (to be added) 1.b) Interm

Re: [PATCH] Add -fdump-rtl--quiet

2012-04-18 Thread Xinliang David Li
On Wed, Apr 18, 2012 at 12:42 PM, Andrew Stubbs wrote: > On 18/04/12 19:00, Xinliang David Li wrote: >> >> Why only rtl? Minor suggestion: use ir or il may be more intuitive: >> -fdump-rtl-all-ir > > > It isn't only RTL. It also applies to the tree dumps. I did say so in my > message, although all

Re: [PATCH] Add -fdump-rtl--quiet

2012-04-18 Thread Andrew Stubbs
On 18/04/12 19:00, Xinliang David Li wrote: Why only rtl? Minor suggestion: use ir or il may be more intuitive: -fdump-rtl-all-ir It isn't only RTL. It also applies to the tree dumps. I did say so in my message, although all the examples were RTL. Also, "ir" and "il" are not more intuitive t

Re: [PATCH] Add -fdump-rtl--quiet

2012-04-18 Thread Xinliang David Li
Why only rtl? Minor suggestion: use ir or il may be more intuitive: -fdump-rtl-all-ir thanks, David On Wed, Apr 18, 2012 at 9:35 AM, Andrew Stubbs wrote: > This patch scratches an itch I've had for a while. > > Basically it just reduces all tree and RTL dumps to just the function dump. > This m

[PATCH] Add -fdump-rtl--quiet

2012-04-18 Thread Andrew Stubbs
This patch scratches an itch I've had for a while. Basically it just reduces all tree and RTL dumps to just the function dump. This means that the files are easier to diff. This can still be combined with the other options of course: e.g. -fdump-rtl-all-quiet-slim is quite nice. Obviously,