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

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 x86_64/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