[lilliput-jdk17u:lilliput] Integrated: 8308641: [Lilliput/JDK17] Remove wrong assert in klass.hpp
Aleksey Shipilev
shade at openjdk.org
Tue May 23 15:27:45 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
Marked as reviewed by shade (Reviewer).
-------------
PR Review: https://git.openjdk.org/lilliput-jdk17u/pull/23#pullrequestreview-1439432486
More information about the lilliput-dev
mailing list