On Mon, 27 May 2024 12:24:31 GMT, Chen Liang <li...@openjdk.org> wrote:
>> Adam Sotona has updated the pull request incrementally with one additional >> commit since the last revision: >> >> performance improvements > > src/java.base/share/classes/java/lang/reflect/ProxyGenerator.java line 822: > >> 820: .iconst_0() // false >> 821: .aload(0)// classLoader >> 822: .invokestatic(CD_Class, "forName", >> MTD_Class_String_boolean_ClassLoader); > > We can probably replace this `forName(name, false, thisClassLoader)` with > loading a class constant to reduce load on symbols. I'm wondering why all the `Class.forName(... <proxyClass>.getClassLoader())` is necessary? Simple `ldc <class entry>` seems to work well. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/19410#discussion_r1616232095
