[lilliput-jdk17u:lilliput] Integrated: 8308641: [Lilliput/JDK17] Remove wrong assert in klass.hpp

Roman Kennke rkennke at openjdk.org
Tue May 23 15:27:50 UTC 2023


On Tue, 23 May 2023 11:21:16 GMT, Roman Kennke <rkennke at openjdk.org> wrote:

> In klass.hpp, Klass::prototype_header() we have an assert(UseCompactObjectHeaders) which came from the similar upstream jdk21 change, but is wrong in jdk17, because the prototype_header() will also be used when biased locking is used. Biased locking is disabled by default, and the assert is harmless in release builds, but it makes a bunch of tests fail. Let's remove the assert and reduce upstream-jdk17u diff a little bit.
> 
> Testing:
>  - [x] tier1 on x86, where some tests failed before

This pull request has now been integrated.

Changeset: 7aba2d26
Author:    Roman Kennke <rkennke at openjdk.org>
URL:       https://git.openjdk.org/lilliput-jdk17u/commit/7aba2d2697459a89a96fe914693a8375db870485
Stats:     4 lines in 1 file changed: 0 ins; 3 del; 1 mod

8308641: [Lilliput/JDK17] Remove wrong assert in klass.hpp

Reviewed-by: shade

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

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


More information about the lilliput-dev mailing list