[master] Integrated: 8307991: [Lilliput] Use narrow i-hash only when +UseCompactObjectHeaders

Roman Kennke rkennke at openjdk.org
Fri May 12 16:05:23 UTC 2023


On Fri, 12 May 2023 10:33:45 GMT, Roman Kennke <rkennke at openjdk.org> wrote:

> Currently, we use a narrow 25bit identity-hash-code in Lilliput. That is even the case when running without compact object headers, even though we could just as well use the original 31bit identity hash-code then.
> 
> The fix is relatively simple. Also, this brings the code in line with what we're currently proposing in upstream https://github.com/openjdk/jdk/pull/13844.
> 
> Going forward we would trade Klass* bits to get the full 31 i-hash back, and we're going to do that soon, but let's first fix this issue at hand (also in preparation for a clean backport, where we don't currently have the metaspace improvements).
> 
> Testing:
>  - [x] tier1 +UseCompactObjectHeaders
>  - [x] tier2 +UseCompactObjectHeaders
>  - [x] tier1 -UseCompactObjectHeaders
>  - [x] tier2 -UseCompactObjectHeaders

This pull request has now been integrated.

Changeset: 646f6f5e
Author:    Roman Kennke <rkennke at openjdk.org>
URL:       https://git.openjdk.org/lilliput/commit/646f6f5e2ba276bb510ea6460232026d46cede53
Stats:     58 lines in 7 files changed: 42 ins; 0 del; 16 mod

8307991: [Lilliput] Use narrow i-hash only when +UseCompactObjectHeaders

Reviewed-by: shade

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

PR: https://git.openjdk.org/lilliput/pull/93


More information about the lilliput-dev mailing list