RFR: 8349539: [leyden] Allow dumping nmethods that reference archived hidden klasses [v4]
Aleksey Shipilev
shade at openjdk.org
Tue Feb 11 16:40:30 UTC 2025
On Thu, 6 Feb 2025 22:00:38 GMT, Vladimir Ivanov <vlivanov at openjdk.org> wrote:
>> 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
>
> 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.
Are we good with this, @iwanowww? Is this the correct place to move the check?
-------------
PR Comment: https://git.openjdk.org/leyden/pull/33#issuecomment-2651371189
More information about the leyden-dev
mailing list