Re: [PATCH] Fix PR ipa/65432

[Martin LiÅka <mliska at suse dot cz>]

On 03/18/2015 03:05 PM, Jakub Jelinek wrote:
> On Wed, Mar 18, 2015 at 02:59:10PM +0100, Martin LiÅka wrote:
> > >From 5d2e883524ba23155d80cd4ad6d58ba5c73a1f90 Mon Sep 17 00:00:00 2001
> > From: mliska <mliska@suse.cz>
> > Date: Wed, 18 Mar 2015 13:59:49 +0100
> > Subject: [PATCH] Fix PR ipa/65432
> >
> > gcc/ChangeLog:
> >
> > 2015-03-18  Martin Liska  <mliska@suse.cz>
>
> Missing PR ipa/65432 here...
>
> > 	* cgraph.c (cgraph_node::get_create): Remove unnecessary
> > 	xstrdup_for_dump wrapper.
> > 	* ipa-icf.c (sem_item::dump): Use symtab_node::name instead of
> > 	sem_item::name.
> > 	(sem_function::equals): Wrap symtab_node::name and symtab_node::asm_name
> > 	with xstrdup_for_dump.
> > 	(sem_variable::equals): Likewise.
> > 	(sem_item_optimizer::read_section): Use symtab_node::name instead of
> > 	sem_item::name.
> > 	(sem_item_optimizer::parse_funcs_and_vars): Likewise.
> > 	(sem_item_optimizer::merge_classes): Wrap symtab_node::name and
> > 	symtab_node::asm_name with xstrdup_for_dump.
> > 	(congruence_class::dump): Use symtab_node::name instead of
> > 	sem_item::name.
> > 	* ipa-icf.h (symtab_node::name): Remove.
> > 	(symtab_node::asm_name): Likewise.
>
> Otherwise LGTM.
>
> 	Jakub

Thank you.

Adding missing comment and going to install the patch.

Martin