On Tue, Apr 23, 2013 at 7:40 AM, Xinliang David Li <davi...@google.com> wrote:
> 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.

Ok.

Thanks,
Richard.

> 2013-04-22  Xinliang David Li  <davi...@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 <dnovi...@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.

Reply via email to