Integrated: 8339492: StackMapDecoder::writeFrames makes lots of allocations

David M. Lloyd duke at openjdk.org
Wed Sep 4 16:51:24 UTC 2024


On Tue, 3 Sep 2024 16:13:39 GMT, David M. Lloyd <duke at openjdk.org> wrote:

> Please review this change, which reduces the number of allocations in `StackMapDecoder::writeFrames` by using a sorted array instead of a `TreeMap<Integer, ...>` to sort and uniquify entries before writing. It also adds a validation missed by the original implementation.

This pull request has now been integrated.

Changeset: 433f6d8a
Author:    David M. Lloyd <david.lloyd at redhat.com>
Committer: Chen Liang <liach at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/433f6d8a0643b59663bf76c0f3a2af27a6cc56b7
Stats:     17 lines in 1 file changed: 8 ins; 1 del; 8 mod

8339492: StackMapDecoder::writeFrames makes lots of allocations

Reviewed-by: liach, redestad, jwaters, asotona

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

PR: https://git.openjdk.org/jdk/pull/20841


More information about the core-libs-dev mailing list