http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60449
Martin Jambor <jamborm at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |hubicka at gcc dot gnu.org
--- Comment #7 from Martin Jambor <jamborm at gcc dot gnu.org> ---
(In reply to Richard Biener from comment #5)
> Shouldn't fixup_cfg fix this up? Simply prefering the decl with attributes
> isn't enough if you attach an urelated attribute to both fns as well.
Yes, I know. I did not really mean to call it a "proposed" fix, did
that somewhat automatically. On the other hand, I am able to slim-LTO
build Firefox with the patch though.
(In reply to Richard Biener from comment #6)
> OTOH, why do we have to merge the decls/cgraph nodes at all? Can't we simply
> make them aliases if tree merging decides the decls are not equal?
Good idea, this might indeed be the best way to fix this. (Although I
do not really know what needs to be done to turn a previously proper
decl and its symtab node into an alias. Honza, do you think it would
be difficult?).