RFR: 8349923: Refactor StackMapTable constructor and StackMapReader

Matias Saavedra Silva matsaave at openjdk.org
Wed Feb 19 17:28:56 UTC 2025


On Fri, 14 Feb 2025 19:32:28 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> This patch refactors the `StackMapReader` to clean up the `StackMapTable` constructor. The `StackMapReader` class is extended to hold more fields used for verification purposes and allows for a simpler StackMapTable constructor. Additionally, in preparation for further changes to verification, the `_frame_count` may not be equal to the value reported by the classfile, so a new structure is used to generate the `_frame_array`. Verified with tier 1-5 tests
>
> Never mind, I do know why frame_count might change with your upcoming changes.

Thanks for the reviews and suggestions @coleenp and @dholmes-ora!

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

PR Comment: https://git.openjdk.org/jdk/pull/23645#issuecomment-2669298522


More information about the hotspot-runtime-dev mailing list