On Mon, 2 Aug 2021 18:27:00 GMT, Claes Redestad <redes...@openjdk.org> wrote:
> Trivial startup hack/cleanup to use the `classLoader` field directly rather > than the `getClassLoader0` accessor. > > Before aa12c8fbf5b020d168cf29412643bf15514b0439 there was no `classLoader` > field and `getClassLoader0` was a native method that retrieved the > `ClassLoader` by calling into the runtime. However, now that `classLoader` is > injected on mirror creation it makes less sense to use the accessor when not > necessary. This looks okay. I assume you concern about the early startup when this method is running in interpreted mode. Do you see any startup improvement by this change? ------------- Marked as reviewed by mchung (Reviewer). PR: https://git.openjdk.java.net/jdk/pull/4960