[master] RFR: OMWorld: Cleanup BasicLock header [v2]
Axel Boldt-Christmas
aboldtch at openjdk.org
Thu Apr 25 05:42:55 UTC 2024
> Cleanup the usage of the header field in BasicLock. Because the asserts are stronger now I also cleanup the zero usage of the BasicLock. Because https://bugs.openjdk.org/browse/JDK-8307532 is not yet implemented I have not tested this interaction but as this now restricts the displaced header in the BasicLock to only LM_LEGACY something had to be done for LM_MONITOR support.
>
> This also changes symbols exposed through JVMCI so it may take extra effort to get this through.
>
> Similarly to #161 some changes could be upstreamed, and here there is already a distinction as neither `LM_LIGHTWEIGHT` nor `LM_MONITOR` uses the displaced header.
Axel Boldt-Christmas has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:
- Merge remote-tracking branch 'upstream_lilliput/master' into lilliput-om-cleanup-basiclock-header
- Fix condition
- BasicLock Restructure (includes JVMCI and zero changes)
-------------
Changes:
- all: https://git.openjdk.org/lilliput/pull/162/files
- new: https://git.openjdk.org/lilliput/pull/162/files/b54babc1..ca28a78a
Webrevs:
- full: https://webrevs.openjdk.org/?repo=lilliput&pr=162&range=01
- incr: https://webrevs.openjdk.org/?repo=lilliput&pr=162&range=00-01
Stats: 76 lines in 5 files changed: 52 ins; 18 del; 6 mod
Patch: https://git.openjdk.org/lilliput/pull/162.diff
Fetch: git fetch https://git.openjdk.org/lilliput.git pull/162/head:pull/162
PR: https://git.openjdk.org/lilliput/pull/162
More information about the lilliput-dev
mailing list