[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