RFR: 8353298: AOT cache creation asserts with _array_klasses in an unregistered InstanceKlass

Calvin Cheung ccheung at openjdk.org
Thu Apr 17 19:58:42 UTC 2025


On Thu, 17 Apr 2025 05:36:41 GMT, Ioi Lam <iklam at openjdk.org> wrote:

> The assert in `Klass::remove_java_mirror()` is too narrow. When creating the AOT cache, it's possible to see two types of Klasses that do not have a Java mirror:
> 
> - Unregistered classes (a cached class to be loaded by a custom class loader in the production run)
> - An array type of an unregistered class
> 
> I have relaxed the assert to accept both conditions

Looks good. Can you add some comment in the test on what it is testing?

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

Marked as reviewed by ccheung (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/24711#pullrequestreview-2776764179


More information about the hotspot-dev mailing list