RFR: 8365191: Cleanup after removing LockingMode related code [v2]
Coleen Phillimore
coleenp at openjdk.org
Wed Sep 24 12:50:50 UTC 2025
On Wed, 24 Sep 2025 11:44:58 GMT, Fredrik Bredberg <fbredberg at openjdk.org> wrote:
>> src/hotspot/share/runtime/vmStructs.cpp line 685:
>>
>>> 683: volatile_nonstatic_field(ObjectMonitor, _owner, int64_t) \
>>> 684: volatile_nonstatic_field(ObjectMonitor, _next_om, ObjectMonitor*) \
>>> 685: volatile_nonstatic_field(BasicLock, _monitor, ObjectMonitor*) \
>>
>> Since nothing now refers to this, you can delete it from vmStructs.
>
> According to @mur47x111, they still need this line for their fast locking implementation.
Oh you were supposed to leave the field in vmStructs_jvmci.cpp and remove it from this one.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27448#discussion_r2375681060
More information about the serviceability-dev
mailing list