RFR: 8228604: StackMapFrames are missing from redefined class bytes of retransformed classes

Tue Jan 24 03:16:06 UTC 2023

On Tue, 24 Jan 2023 00:16:10 GMT, Alex Menkov <amenkov at openjdk.org> wrote:

> This code is old (from "initial load") and I don't understand the reason it was implemented this way.

My guess is footprint reduction. The assumption was probably that the stackmaps are not needed if the method does not need verification. You might want to consider retaining this footprint reduction for minimalVM. Basically make it conditional on INCLUDE_JVMTI, but I'm ok with the changes as-is also.

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

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