Review request for 6807702 -- approved

Joseph D. Darcy Joe.Darcy at Sun.COM
Fri Mar 20 17:02:55 UTC 2009


Looks good -- approved.

-Joe

Alan Bateman wrote:
>
> 6807702: Integer.valueOf cache should be configurable
>
> This is a forward-port from 6u14, which in turn is an update to the 
> original library updates that Tom Rodriguez did in an earlier 6 
> update. Joe, you'll probably want to review this one as it relates to 
> the cache you developed back in jdk5. The background to this is work 
> in HotSpot to eliminate the use of box objects. The optimizer knows 
> about Integer's autobox cache since hs11. Further performance can be 
> gained by increasing the size of the cache, which HotSpot does when 
> running with +AggressiveOpts.
>
> The webrev is here:
>  http://cr.openjdk.java.net/~alanb/6807702/webrev.00/
>
> Thanks,
>
> Alan.




More information about the core-libs-dev mailing list