This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Improve -fprofile-report.
- From: Martin Liška <mliska at suse dot cz>
- To: gcc-patches at gcc dot gnu dot org
- Cc: Richard Biener <richard dot guenther at gmail dot com>
- Date: Tue, 13 Nov 2018 08:51:15 +0100
- Subject: Re: [PATCH] Improve -fprofile-report.
- References: <80fd0395-865c-3c25-523f-a5d1342a453a@suse.cz>
PING^1
On 11/6/18 3:05 PM, Martin Liška wrote:
> Hi.
>
> The patch is based on what was discussed on IRC and in the PR.
> Apart from that the reported layout is improved.
>
> Patch survives regression tests on x86_64-linux-gnu.
>
> Ready for trunk?
> Martin
>
> gcc/ChangeLog:
>
> 2018-11-06 Martin Liska <mliska@suse.cz>
>
> PR tree-optimization/87885
> * cfghooks.c (account_profile_record): Rename
> to ...
> (profile_record_check_consistency): ... this.
> Calculate missing num_mismatched_freq_in.
> (profile_record_account_profile): New function
> that calculates time and size of a function.
> * cfghooks.h (struct profile_record): Remove
> all tuples.
> (struct cfg_hooks): Remove after_pass flag.
> (account_profile_record): Rename to ...
> (profile_record_check_consistency): ... this.
> (profile_record_account_profile): New.
> * cfgrtl.c (rtl_account_profile_record): Remove
> after_pass flag.
> * passes.c (check_profile_consistency): Do only
> checking.
> (account_profile): Calculate size and time of
> function only.
> (pass_manager::dump_profile_report): Reformat
> output.
> (execute_one_ipa_transform_pass): Call
> consistency check before clean upand call account_profile
> after a clean up is done.
> (execute_one_pass): Call check_profile_consistency and
> account_profile instead of using after_pass flag..
> * tree-cfg.c (gimple_account_profile_record): Likewise.
> ---
> gcc/cfghooks.c | 38 +++++++--
> gcc/cfghooks.h | 17 ++--
> gcc/cfgrtl.c | 12 ++-
> gcc/passes.c | 207 ++++++++++++++++++++++++++-----------------------
> gcc/tree-cfg.c | 11 ++-
> 5 files changed, 161 insertions(+), 124 deletions(-)
>
>