RFR: 8349539: [leyden] Allow dumping nmethods that reference archived hidden klasses [v4]

Vladimir Ivanov vlivanov at openjdk.org
Tue Feb 11 19:16: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

Looks good.

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

Marked as reviewed by vlivanov (Committer).

PR Review: https://git.openjdk.org/leyden/pull/33#pullrequestreview-2609783931


More information about the leyden-dev mailing list