On Fri, 11 Dec 2015, Jan Hubicka wrote: > > > > We explicitely do not use debug-info-level tests in free-lang-data > > to allow mixing -g and -g0 objects. Are you sure doing the above > > doesn't mess up tree merging enough to effectively enlarge WPA > > memory use and the merged decl sections? > > > > [I'm quite sure firefox build system manages to mess up -g vs. -g0 > > in some places ;)] > > Hmm, I will try the debug build with firefox on this. -fdump-ipa-devirt > now dumps all main variants that are duplicates of one ODR type. > We definitely have types with hundreds of duplicates, so there are > quite common cases where tree merging does not fire. > > > > > + return (!DECL_IGNORED_P (decl) && !is_redundant_typedef (decl)); > > > +} > > > + > > > > The patch would be ok if you simply export is_redundant_typedef > > and inline the DECL_IGNORED_P check into free-lang-data. > > OK, I had that originally, will return that back. > is_redundant_typedef is declared inline. Putting it to tree.h drags > bit too many dwarf2out internals, but I suppose it is OK to just > turn it non-inline. It is a type of function where inliner should be > able to decide.
Yeah. Richard. > Honza > > > > Thanks, > > Richard.
