On Mon, 12 Aug 2024 18:55:41 GMT, Coleen Phillimore <cole...@openjdk.org> wrote:
>> Axel Boldt-Christmas has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Missing DEBUG_ONLY > > src/hotspot/share/runtime/lightweightSynchronizer.cpp line 341: > >> 339: }; >> 340: >> 341: ObjectMonitorTable* LightweightSynchronizer::_omworld = nullptr; > > I preferred my version where ObjectMonitorTable was AllStatic which gets rid > of _omworld-> everwhere, but the internal table is the pointer. You can > remove more omworld names also. I implemented the AllStatic table, and remove all mentions of OMWorld. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/20067#discussion_r1715432994