On 11/14/2017 05:07 PM, Martin Liška wrote:
> On 11/14/2017 10:20 AM, Jan Hubicka wrote:
>> @@ -4670,11 +4671,12 @@ expand_call_inline (basic_block bb, gimp
>> if (dump_file && (dump_flags & TDF_DETAILS))
>> {
>> - fprintf (dump_file, "Inlining ");
>> - print_generic_expr (dump_file, id->src_fn);
>> - fprintf (dump_file, " to ");
>> - print_generic_expr (dump_file, id->dst_fn);
>> - fprintf (dump_file, " with frequency %i\n", cg_edge->frequency ());
>> + fprintf (dump_file, "Inlining %s to %s with frequency %4.2f\n",
>> + xstrdup_for_dump (id->src_node->dump_name ()),
>> + xstrdup_for_dump (id->dst_node->dump_name ()),
>> + cg_edge->sreal_frequency ().to_double ());
>> + id->src_node->dump (dump_file);
>> + id->dst_node->dump (dump_file);
>> }
>
> Hi.
>
> You don't have to call xstrdup_for_dump for functions symtab_node::dump_name
> and
> symtab_node::dump_asm_name. Allocation is done from GGC memory.
>
> Martin
Tested and installed as r255005.
Martin
