RFR: 8349539: [leyden] Allow dumping nmethods that reference archived hidden klasses [v3]
Vladimir Kozlov
kvn at openjdk.org
Mon Feb 10 15:19:43 UTC 2025
On Mon, 10 Feb 2025 10:04:28 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 three additional commits since the last revision:
>
> - Merge branch 'premain' into JDK-8349539-allow-hidden
> - Moving the check around
> - Fix
Looks good to me.
-------------
Marked as reviewed by kvn (Committer).
PR Review: https://git.openjdk.org/leyden/pull/33#pullrequestreview-2606180079
More information about the leyden-dev
mailing list