This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Final intermodule patch
Jan Hubicka <jh@suse.cz> writes:
> > Andreas Jaeger <aj@suse.de> writes:
> >
> > > Mike Stump <mrs@apple.com> writes:
> > >
> > > > On Friday, July 11, 2003, at 03:36 AM, Andi Kleen wrote:
> > > >> Can you expand a bit on how usable this is yet?
> > > >
> > > > Should be fairly usable, though, it hasn't seen tons of use. It does
> > > > all of SPEC (well, except eon) for example, just to give you a flavor
> > > > of what has already been though it.
> > >
> > > Do you have any SPEC numbers that you can share? How much improvement
> > > does it bring?
> >
> > If you just use this patch, you get a decrease in SPEC performance,
> > because GCC's current inlining heuristics don't work very well in that
> > situation. With improved heuristics (Dale's and Stuart's work), we
> > got significant improvement instead of a decrease.
>
> What kind of improvements have you implemented? How does it relate to
> the new inlining heuristics code I commited last week?
The most significant part is feedback-directed inlining, which is a
big win, as you might expect since it suppresses the existing
heuristics and uses actual feedback data. Unfortunately, as I
understand it, your new code makes our changes need a rewrite, so it
might be some time before we can get them into FSF GCC. There are
also some unsolved issues, like how to do feedback-directed inlining
at the same time as effective feedback-directed branch prediction.
--
- Geoffrey Keating <geoffk@geoffk.org>