RFR (S) CR 8016236: Class.getGenericInterfaces performance improvement (original) (raw)

Alan Bateman Alan.Bateman at oracle.com
Mon Jun 17 13:22:15 UTC 2013

On 17/06/2013 13:32, Aleksey Shipilev wrote:

On 06/17/2013 03:44 PM, Alan Bateman wrote:

This looks to good. I agree with Peter's point about the 2 reads in getInterfaces() although it's unlikely to make a significant difference. Oh, it can have the difference on non-x86 hardware and/or in optimized code. This nit is fixed here: http://cr.openjdk.java.net/~shade/8016236/webrev.04/ I did the same testing as before: Linux x8664/release builds and runs jdk/test/java/lang/reflect jtreg successfully; the performance is still there.

If you can create the change-set then I can push this for you today. Here: http://cr.openjdk.java.net/~shade/8016236/8016236.changeset Although only Alan counts as official reviewer; do we need another one? The updated webrev looks good for me.

For the jdk repository then only one reviewer with "reviewer" role is required. I see you have Peter and Paul listed in a Reviewed-by line so I think we are good. Do you want Doug listed too as this started out as a patch from Doug.

-Alan

More information about the core-libs-dev mailing list