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.

Reply via email to