Review request for 6807702 -- approved (original) (raw)

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