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

Vladimir Ivanov vlivanov at openjdk.org
Thu Feb 6 22:03:47 UTC 2025


On Thu, 6 Feb 2025 11:52:25 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.

I agree that the restriction for hidden classes can be lifted.

But there are non-default modes where it still matters (`UseMetadataPointers == true`). 
I suggest to keep `klass->is_hidden()` check, but move it down to guard symbolic resolution case.

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

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


More information about the leyden-dev mailing list