Re: Patch proposal: use __builtin_expect

[Anthony Green <green at cygnus dot com>]

Tom wrote:
> It would be way cool if you ran before-and-after performance
> measurements on this.  I'm really curious to see what the effect is.

I intend to.  I don't know what the diff will be for IA-32.  There are
other architectures where mis-predicting a branch is super expensive.

> Anyway, you can add this macro to jvm.h.  More header work is needed...

D'oh! Of course.  I was thinking that jvm.h was installed.

> I like this patch.  I have no idea how you found all those places to
> put __builtin_expect.  Cool.

I remember the commonly called functions from my profiling a few weeks
ago - and then I just searched for "if (".  :-)

AG

-- 
Anthony Green                                                        Red Hat
                                                       Sunnyvale, California