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: Xinliang David Li <davidxl at google dot com>
- To: Diego Novillo <dnovillo at google dot com>
- Cc: Steven Bosscher <stevenb dot gcc at gmail dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Mon, 22 Apr 2013 22:40:43 -0700
- 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> <CABu31nOTvmpEQse+Bdwz-SvyuzCOE_BqdVm-HUnRZ0bEcr5=pg at mail dot gmail dot com> <CAAkRFZJqZMOm7YWvfovKBMnENAJf9-nSg=zW68MDN1CgC1g41Q at mail dot gmail dot com> <51758E2B dot 7040000 at google dot com>
Hi, this is a follow up simple patch to support 'slim' graph dump:
when -slim option is specified (e.g,
-fdump-tree-optimized-graph-slim), do not dump the boby.
2013-04-22 Xinliang David Li <davidxl@google.com>
* cfghhooks.c (dump_bb_for_graph): Support 'slim' graph dump.
Index: cfghooks.c
===================================================================
--- cfghooks.c (revision 198165)
+++ cfghooks.c (working copy)
@@ -313,7 +313,8 @@ dump_bb_for_graph (pretty_printer *pp, b
pp_printf (pp, "COUNT:" HOST_WIDEST_INT_PRINT_DEC, bb->count);
pp_printf (pp, " FREQ:%i |", bb->frequency);
pp_write_text_to_stream (pp);
- cfg_hooks->dump_bb_for_graph (pp, bb);
+ if (!(dump_flags & TDF_SLIM))
+ cfg_hooks->dump_bb_for_graph (pp, bb);
}
/* Dump the complete CFG to FILE. FLAGS are the TDF_* flags in dumpfile.h. */
Ok for trunk?
thanks,
David
On Mon, Apr 22, 2013 at 12:23 PM, Diego Novillo <dnovillo@google.com> wrote:
> On 2013-04-21 02:37 , Xinliang David Li wrote:
>>
>> * graph.c (draw_cfg_node_succ_edges): Add branch probility as
>> label.
>> * cfghhooks.c (dump_bb_for_graph): Dump profile count and
>> frquency.
>> * Makefile.in: New dependency.
>
>
> Looks OK.
>
>
> Diego.