[lilliput-jdk17u:lilliput] Integrated: 8310558: [Lilliput/JDK17] Missing klass gap initialization in ContiguousSpace::allocate_temporary_filler

[Aleksey Shipilev]

On Wed, 21 Jun 2023 15:39:35 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

> Minor thing, but still.
> 
> This is Lilliput JDK 17 specific, later versions have [JDK-8278568](https://bugs.openjdk.org/browse/JDK-8278568) refactoring.
> 
> We have missed the `set_klass_gap(0)` line in the patch here:
> 
> 
>     } else {
>       assert(size == CollectedHeap::min_fill_size(),
>              "size for smallest fake object doesn't match");
>       instanceOop obj = (instanceOop) cast_to_oop(allocate(size));
> - obj->set_mark(markWord::prototype());
> - obj->set_klass_gap(0);
> - obj->set_klass(vmClasses::Object_klass());
> + if (UseCompactObjectHeaders) {
> + obj->set_mark(vmClasses::Object_klass()->prototype_header());
> + } else {
> + obj->set_mark(markWord::prototype());
> + obj->set_klass(vmClasses::Object_klass());
> + }
>     }
>   }
> 
> 
> I had to restore the klass gap methods in `oop`, which also limits the impact we have on upstream code.

This pull request has now been integrated.

Changeset: 1d73ede3
Author:    Aleksey Shipilev <shade at openjdk.org>
URL:       https://git.openjdk.org/lilliput-jdk17u/commit/1d73ede33e9a79e20b5175d24abc50ef2fcee4fd
Stats:     13 lines in 3 files changed: 13 ins; 0 del; 0 mod

8310558: [Lilliput/JDK17] Missing klass gap initialization in ContiguousSpace::allocate_temporary_filler

Reviewed-by: rkennke

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

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