On Mon, 27 Mar 2023 15:07:42 GMT, Adam Sotona <[email protected]> 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