This is the mail archive of the java-patches@sourceware.cygnus.com mailing list for the Java project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: Patch: More __builtin_expect fun



Tom wrote:
> I think we should definitely only do this when there is a measurable
> benefit.  There is a tradeoff with maintainability.

Bryce - did you compile with -freorder-blocks?  I think you need to
use that flag for __builtin_expect to have any effect.  I was never
able to get it working (there are some general compiler issues to sort
out still - I think an important fix went in today).  

Admittedly, my __builtin_expect patch was partially speculative
although I did have some profile info to guide me.

AG

-- 
Anthony Green                                                        Red Hat
                                                       Sunnyvale, California

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]