RFR: 8274858: Remove unused dictionary_classes_do functions

David Holmes dholmes at openjdk.java.net
Thu Oct 7 02:02:04 UTC 2021


On Wed, 6 Oct 2021 20:35:21 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

> These were needed once by JVMTI to find classes that have been initiated by a class loader and not just defined, but that code was changed.  Other callers have been removed. The remaining use doesn't need initiated classes for initializing itables, so doesn't need to walk the dictionary only the ClassLoaderData as the defining class loader.
> 
> In this place, only the bootstrap loader is loaded, so the CLDG walk is overkill but this is only for !UseSharedSpaces so harmless and prevents making functions public.
> 
> Tested with tier1-3

Hi Coleen,

This seems fine.

Thanks,
David

-------------

Marked as reviewed by dholmes (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/5843


More information about the hotspot-runtime-dev mailing list