On Thu, 21 Dec 2023 01:12:07 GMT, Adam Sotona <asot...@openjdk.org> wrote:
>> java.base java.lang.reflect.ProxyGenerator uses ASM to generate proxy >> classes. >> >> This patch converts it to use Classfile API. >> >> It is continuation of https://github.com/openjdk/jdk/pull/10991 >> >> Any comments and suggestions are welcome. >> >> Please review. >> >> Thank you, >> Adam > > Adam Sotona has updated the pull request incrementally with two additional > commits since the last revision: > > - performance improvements > - SplitConstantPool performance fix src/java.base/share/classes/jdk/internal/classfile/impl/StackCounter.java line 62: > 60: private final Utf8Entry methodDesc; > 61: private final SplitConstantPool cp; > 62: private final LinkedList<Target> targets; Wouldn't it be better to use `ArrayDeque` instead of `LinkedList`? ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/17121#discussion_r1433979344