[lilliput-jdk17u:lilliput] Integrated: 8310944: [Lilliput/JDK17] Streamline and cleanup GC code

Roman Kennke rkennke at openjdk.org
Fri Jun 30 10:27:25 UTC 2023


On Tue, 27 Jun 2023 11:31:43 GMT, Roman Kennke <rkennke at openjdk.org> wrote:

> We have a bunch of ugly hacks in GC code paths where we do stuff like:
> if (UseCompactObjectHeaders) {
>  // fetch object Klass* or size in a safe way...
> } else {
>   // fetch object Klass* or size in the traditional way
> }
> 
> This happens in various places in similar ways. I propose to add relevant methods to oopDesc, and make all GC code use those new methods instead.
> 
> This helps streamline and clean up a bunch of GC code, and bring it closer to the JDK22 upstreaming PR.
> 
> Testing:
>  - [x] hotspot_gc
>  - [x] hotspot_gc (+UCOH)
>  - [x] tier1
>  - [x] tier1 (+ShenandoahGC)
>  - [x] tier1 (+ParallelGC)
>  - [x] tier1 (+SerialGC)
>  - [x] tier1 (+UCOH)
>  - [x] tier1 (+UCOH +ShenandoahGC)
>  - [x] tier1 (+UCOH +ParallelGC)
>  - [x] tier1 (+UCOH +SerialGC)
>  - [x] tier2
>  - [x] tier2 (+UCOH)

This pull request has now been integrated.

Changeset: 723d034d
Author:    Roman Kennke <rkennke at openjdk.org>
URL:       https://git.openjdk.org/lilliput-jdk17u/commit/723d034d81518fe6e81eb3299f8706912de50c81
Stats:     301 lines in 16 files changed: 77 ins; 180 del; 44 mod

8310944: [Lilliput/JDK17] Streamline and cleanup GC code

Reviewed-by: shade

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

PR: https://git.openjdk.org/lilliput-jdk17u/pull/50


More information about the lilliput-dev mailing list