This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Simple change in dot dumper -- Display profile count and branch probability
- From: Steven Bosscher <stevenb dot gcc at gmail dot com>
- To: Xinliang David Li <davidxl at google dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Sun, 21 Apr 2013 01:02:10 +0200
- Subject: Re: Simple change in dot dumper -- Display profile count and branch probability
- References: <CAAkRFZKApEq_vf1g_C-OejkiQjVNZXdye4JxxTMdtA+CkjGHOA at mail dot gmail dot com>
On Sun, Apr 21, 2013 at 12:47 AM, Xinliang David Li wrote:
> Index: graph.c
> ===================================================================
> --- graph.c (revision 198108)
> +++ graph.c (working copy)
> @@ -110,6 +110,9 @@ draw_cfg_node (pretty_printer *pp, int f
> else
> {
> pp_character (pp, '{');
> + if (bb->count)
> + pp_printf (pp, "COUNT:" HOST_WIDEST_INT_PRINT_DEC, bb->count);
> + pp_printf (pp, " FREQ:%i |", bb->frequency);
> pp_write_text_to_stream (pp);
> dump_bb_for_graph (pp, bb);
> pp_character (pp, '}');
This doesn't belong here, please put it in cfghooks.c:dump_bb_for_graph.
Otherwise, looks good me.
Ciao!
Steven