RFR: 8349539: [leyden] Allow dumping nmethods that reference archived hidden klasses [v4]
duke
duke at openjdk.org
Tue Feb 11 19:22:28 UTC 2025
On Mon, 10 Feb 2025 16:34:02 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
>> Studying the bailout reasons for JavacBenchApp, I notices the majority of nmethod store bailouts are due to referencing the hidden klasses. Those checks seem to be from the initial commit. I wonder if we can actually skip hidden klass checks, now that we archive lots of those since invokedynamic AOT was implemented.
>
> Aleksey Shipilev has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains four additional commits since the last revision:
>
> - Merge branch 'premain' into JDK-8349539-allow-hidden
> - Merge branch 'premain' into JDK-8349539-allow-hidden
> - Moving the check around
> - Fix
@shipilev
Your change (at version b4ec97aca8cbc2ef33fdcf058b7ed2daad65c69c) is now ready to be sponsored by a Committer.
-------------
PR Comment: https://git.openjdk.org/leyden/pull/33#issuecomment-2651848026
More information about the leyden-dev
mailing list