> On 07/15/2014 04:46 AM, Jan Hubicka wrote:
> >I think we still want to check that the local comdats are linked into
> >the corresponding comdat group, so we probably want
> >to test node->definition instead of node->public, or perhaps just clear
> >COMDAT_GROUP info when removing symbol?
> 
> Like this?
> 
> 

> commit deb1a9e023d457c88edf52231d32af5ef03179f4
> Author: Jason Merrill <ja...@redhat.com>
> Date:   Mon Jul 14 16:58:57 2014 -0400
> 
>       PR c++/61623
>       * symtab.c (symtab_remove_from_same_comdat_group): Also
>       set_comdat_group to NULL_TREE.
>       (verify_symtab): Fix diagnostic.

Yes, this looks perfect. Thanks a lot for looking into this!
Honza

Reply via email to