RFR (M) 8198313: Wrap holder object for ClassLoaderData in a WeakHandle

Kim Barrett kim.barrett at oracle.com
Sat Mar 31 18:40:13 UTC 2018


> On Mar 30, 2018, at 1:53 PM, coleen.phillimore at oracle.com wrote:
> 
> 
> I have an incremental and full .02 version with the changes discussed here.
> 
> open webrev at http://cr.openjdk.java.net/~coleenp/8198313.02.incr/webrev
> open webrev at http://cr.openjdk.java.net/~coleenp/8198313.02/webrev
> 
> These have been retested on x86, all hotspot jtreg tests.
> thanks,
> Coleen

Looks good.

In InstanceKlass::klass_holder_phantom, the klass_ prefix seems
unnecessary. I don't need a new webrev if you decide to change the
name.



More information about the hotspot-dev mailing list