On Mon, 27 Mar 2023 15:07:42 GMT, Adam Sotona <asot...@openjdk.org> wrote:

>> src/java.base/share/classes/jdk/internal/classfile/impl/StackMapGenerator.java
>>  line 256:
>> 
>>> 254: 
>>> 255:     private Frame getFrame(int offset) {
>>> 256:         for (var f : frames) {
>> 
>> How large is the frames list expected to be? We can probably perform binary 
>> searches if the list is too large.
>
> The number of frames is usually low, however I'll try to benchmark the 
> difference.
> Thanks.

For ProxyGenerator the difference was insignificant, however in benchmarks 
transforming also more complex code it improved StackMapGenerator performance 
by approx. 7%.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/10991#discussion_r1149520090

Reply via email to