Re: Fix LTO libreofice build

2015-02-17 Thread Jakub Jelinek
Hi! On Tue, Feb 17, 2015 at 09:50:30AM +0100, Jan Hubicka wrote: > * varasm.c (default_binds_local_p_2): External definitions do not > count as definitions here. That is PR65074, but rth prefers a different fix in that area. > --- varasm.c (revision 220741) > +++ varasm.c (working

Fix LTO libreofice build

2015-02-17 Thread Jan Hubicka
Hi, this patch fixes linker error seen during LTO build of libmerged. Linker complains that symbol resolved to dynamic library is hidden and therefore should be defined locally. The bug is already in LTO symbol table that calls binds_local_p and uses hidden visibility on those symbols and in this