RFR 8209821: Make JVMTI GetClassLoaderClasses not walk CLDG

Lois Foltan lois.foltan at oracle.com
Fri Aug 24 15:44:57 UTC 2018

On 8/23/2018 8:37 AM, coleen.phillimore at oracle.com wrote:

> Summary: And also added function with KlassClosure to remove the hacks.
> There are about 10 vmTestbase/nsk/jvmti tests that test various parts 
> of this change.  Also ran mach5 tier1-7.
> open webrev at http://cr.openjdk.java.net/~coleenp/8209821.01/webrev
> bug link https://bugs.openjdk.java.net/browse/JDK-8209821
> Thanks,
> Coleen

Hi Coleen,

I think this is a good clean up.  Couple of comments.

- memory/universe.cpp
You could make basic_type_classes_do() be a for loop
     for (int i = 0; i < T_VOID+1; i++) {

- prims/jvmtiGetLoadedClasses.cpp
In JvmtiGetLoadedClasses::getClassLoaderClasses() you could pull the 
call to basic_type_classes_do() from both sections of the if statement 
to line #139


More information about the hotspot-runtime-dev mailing list