Re: [PATCH][arm] Add -mverbose-cost-dump and de-verbosify cost dumps

2017-12-18 Thread Kyrill Tkachov
On 18/12/17 04:09, Sandra Loosemore wrote: On 12/14/2017 10:50 PM, Sandra Loosemore wrote: On 12/14/2017 11:48 AM, Kyrill Tkachov wrote: [snip] Thanks, done. I haven't created a new Target-specific developers options table but instead listed the targets the options apply to in parentheses.

Re: [PATCH][arm] Add -mverbose-cost-dump and de-verbosify cost dumps

2017-12-17 Thread Sandra Loosemore
On 12/14/2017 10:50 PM, Sandra Loosemore wrote: On 12/14/2017 11:48 AM, Kyrill Tkachov wrote: [snip] Thanks, done. I haven't created a new Target-specific developers options table but instead listed the targets the options apply to in parentheses. Attached is the latest iteration. Thank yo

Re: [PATCH][arm] Add -mverbose-cost-dump and de-verbosify cost dumps

2017-12-14 Thread Sandra Loosemore
On 12/14/2017 11:48 AM, Kyrill Tkachov wrote: [snip] Thanks, done. I haven't created a new Target-specific developers options table but instead listed the targets the options apply to in parentheses. Attached is the latest iteration. Thank you for the review, Kyrill 2017-12-14  Kyrylo Tkach

Re: [PATCH][arm] Add -mverbose-cost-dump and de-verbosify cost dumps

2017-12-14 Thread Kyrill Tkachov
On 14/12/17 18:48, Kyrill Tkachov wrote: On 14/12/17 18:23, Sandra Loosemore wrote: > On 12/14/2017 10:56 AM, Kyrill Tkachov wrote: >> >> 2017-12-14 Kyrylo Tkachov >> >> * doc/invoke.texi (GCC Developer options): Add Target-specific >> developer options subsection. Populate it with

Re: [PATCH][arm] Add -mverbose-cost-dump and de-verbosify cost dumps

2017-12-14 Thread Kyrill Tkachov
On 14/12/17 18:23, Sandra Loosemore wrote: On 12/14/2017 10:56 AM, Kyrill Tkachov wrote: 2017-12-14 Kyrylo Tkachov * doc/invoke.texi (GCC Developer options): Add Target-specific developer options subsection. Populate it with AArch64 and ARM options. > diff --git a/gcc/doc

Re: [PATCH][arm] Add -mverbose-cost-dump and de-verbosify cost dumps

2017-12-14 Thread Sandra Loosemore
On 12/14/2017 10:56 AM, Kyrill Tkachov wrote: 2017-12-14  Kyrylo Tkachov      * doc/invoke.texi (GCC Developer options): Add Target-specific     developer options subsection.  Populate it with AArch64 and ARM     options. > diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi index 0d56

Re: [PATCH][arm] Add -mverbose-cost-dump and de-verbosify cost dumps

2017-12-14 Thread Kyrill Tkachov
Hi Sandra, On 14/12/17 17:09, Sandra Loosemore wrote: On 12/14/2017 08:53 AM, Kyrill Tkachov wrote: Hi all, This patch adds an -mverbose-cost-dump option, similar to the one in aarch64. It makes the RTX cost dump print the RTX we're costing in the backend, as well as its cost. This can be d

Re: [PATCH][arm] Add -mverbose-cost-dump and de-verbosify cost dumps

2017-12-14 Thread Sandra Loosemore
On 12/14/2017 08:53 AM, Kyrill Tkachov wrote: Hi all, This patch adds an -mverbose-cost-dump option, similar to the one in aarch64. It makes the RTX cost dump print the RTX we're costing in the backend, as well as its cost. This can be distracting in other cost-related RTL dumps like combine'

[PATCH][arm] Add -mverbose-cost-dump and de-verbosify cost dumps

2017-12-14 Thread Kyrill Tkachov
Hi all, This patch adds an -mverbose-cost-dump option, similar to the one in aarch64. It makes the RTX cost dump print the RTX we're costing in the backend, as well as its cost. This can be distracting in other cost-related RTL dumps like combine's. So now we don't dump the backend informatio