On Mon, 24 Aug 2026 07:06:29 GMT, Axel Boldt-Christmas <[email protected]> wrote:
>> src/hotspot/share/runtime/objectMonitor.hpp line 236: >> >>> 234: uintptr_t metadata() const; >>> 235: void set_metadata(uintptr_t value); >>> 236: volatile uintptr_t* metadata_addr(); >> >> If I remember this correctly metadata could be two different things, but >> with the lock in the markWord gone, it can now only be one thing. But I >> guess that one thing is the hash code as uintptr_t ? >> Is metadata_addr() used? > > I agree we should probably fold `metadata` into `hash` functions and rename > and retype the field to be `intptr_t _hash` a bit more churn but feels like > we do not benefit from this abstraction anymore. Separate but would also be nice to migrate the object monitors to Atomic. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/32471#discussion_r3841321899
