On Fri, 19 Jun 2026 08:51:35 GMT, Man Cao <[email protected]> wrote:
> Hi all,
>
> Could anyone help review this change that removes the cached static field
> `java.lang.Class.reflectionFactory`? I believe the caching is no longer
> necessary after https://bugs.openjdk.org/browse/JDK-8344011.
>
> This fixes the data race reported in
> https://bugs.openjdk.org/browse/JDK-8386965. It also removes
> `Class.resetArchivedStates()`, which simplifies AOT archive dumping.
>
> -Man
>
> ---------
> - [x] I confirm that I make this contribution in accordance with the [OpenJDK
> Interim AI Policy](https://openjdk.org/legal/ai).
src/java.base/share/classes/java/lang/Class.java line 3395:
> 3393: private static ReflectionFactory getReflectionFactory() {
> 3394: return ReflectionFactory.getReflectionFactory();
> 3395: }
This can probably be replaced with:
import static jdk.internal.reflect.ReflectionFactory.getReflectionFactory;
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/31591#discussion_r3444331312
