On Mon, 15 Jul 2024 00:45:25 GMT, Axel Boldt-Christmas <[email protected]>
wrote:
>> src/hotspot/share/runtime/lightweightSynchronizer.cpp line 477:
>>
>>> 475: if (obj->mark_acquire().has_monitor()) {
>>> 476: if (_length > 0 && _contended_oops[_length-1] == obj) {
>>> 477: //
>>> assert(VM_Version::supports_recursive_lightweight_locking(), "must be");
>>
>> Uncomment or remove assert?
>
> Yeah not sure why it was ever uncommented. To me it seems like that the
> assert should be invariant. But will investigate.
I probably wanted to remove this. It is a tautology on all platforms but arm32
(or other with zero) right now.
So removed it.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20067#discussion_r1713909109
