RFR: 8365501: Remove special AdapterHandlerEntry for abstract methods [v3]

Ashutosh Mehra asmehra at openjdk.org
Tue Sep 2 14:57:52 UTC 2025


On Mon, 1 Sep 2025 10:28:02 GMT, Andrew Dinn <adinn at openjdk.org> wrote:

>> Ashutosh Mehra has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Address review comment - swap conditions
>>   
>>   Signed-off-by: Ashutosh Mehra <asmehra at redhat.com>
>
> src/hotspot/share/runtime/sharedRuntime.cpp line 2567:
> 
>> 2565:   {
>> 2566:     ResourceMark rm;
>> 2567:     MutexLocker mu(AdapterHandlerLibrary_lock);
> 
> Why are we ok to drop this lock here? Was it unnecessary even before this change?

This lock was required because `AdapterHandlerLibrary::create_abstract_method_handler` expected the lock to be held. Now that method is deleted, we don't need to acquire the lock.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/26764#discussion_r2316342459


More information about the hotspot-dev mailing list