This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [RFC] Context sensitive inline analysis
> > The problem is that cgraph_node->uid will be sparse after merging. ?I wonder if we want
> > to solve this by adding new uids to the analyzed nodes that will be denser? Most of summaries
> > are actually attached to the analyzed nodes only.
>
> Can't we re-number the UIDs after merging?
Well, at the moment we read unmerged summaries and merge them later, so it
would man shuffling the per-pass data (that is not big deal to do) but also
won't save peak memory use. The analyzed function uids are by nature always
denser and would have additional advantage of staying more or less dense after
merging (i.e. only comdat funcitons will cause holes). Those will be easilly
used by the inline clones.
Well, I guess I could deffer this for later - once I get class hiarchy on the
callgrpah/varpool, I might pretty much want to have analyzed nodes to inherit
from unanalyzed so we don't have useless data on unanalyzed nodes in general.
Honza
>
> Richard.
>
> > Sadly libxul won't build again for apparently problem:
> > [Leaving LTRANS /abuild/jh/tmp//ccIgav2O.args]
> > [Leaving LTRANS libxul.so.ltrans.out]
> > g++: warning: -pipe ignored because -save-temps specified
> > Reading command line options: libxul.so.ltrans0.olto1: error: ELF section name out of range
> >
> > It seems that for some irrational reason we now decide to stream everything into
> > single partition that is bad idea, but still our ELF infrastructure should not
> > give up.
> > the .o file seems wrong:
> > jh@evans:/abuild/jh/build-mozilla-new11-lto-noelfhackO3/toolkit/library> objdump -h libxul.so.ltrans0.o
> > BFD: libxul.so.ltrans0.o: invalid string offset 4088662 >= 348 for section `.shstrtab'
> > BFD: libxul.so.ltrans0.o: invalid string offset 407 >= 348 for section `(null)'
> > objdump: libxul.so.ltrans0.o: File format not recognized
> >
> >
> > Honza
> >