RFR: 8315884: New Object to ObjectMonitor mapping [v6]

[Axel Boldt-Christmas]

On Mon, 15 Jul 2024 00:45:25 GMT, Axel Boldt-Christmas <aboldtch at openjdk.org> 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