[master] RFR: OMWorld: reenable all platforms

Roman Kennke rkennke at openjdk.org
Wed Jun 12 14:57:38 UTC 2024


On Thu, 23 May 2024 06:26:20 GMT, Axel Boldt-Christmas <aboldtch at openjdk.org> wrote:

> This reenables all platforms to use OMWorld, and by extension UseCompactObjectHeaders. 
> 
> This change simply calls the runtime if a lock is inflated, until port support for OMWorld cache lookup is added.
> 
> ARM (32-bit) required no changes as it already always called the runtime when a monitor is inflated.

Hi Axel, this mostly looks good, I have just one question.

src/hotspot/share/runtime/lightweightSynchronizer.cpp line 1045:

> 1043:   const markWord mark = obj->mark();
> 1044: 
> 1045:   if (mark.is_unlocked()) {

Was that not needed before? Or why is this added now?

-------------

PR Review: https://git.openjdk.org/lilliput/pull/174#pullrequestreview-2113300917
PR Review Comment: https://git.openjdk.org/lilliput/pull/174#discussion_r1636631525


More information about the lilliput-dev mailing list