[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