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

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


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

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

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

Changes: https://git.openjdk.org/lilliput-jdk17u/pull/23/files
 Webrev: https://webrevs.openjdk.org/?repo=lilliput-jdk17u&pr=23&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8308641
  Stats: 4 lines in 1 file changed: 0 ins; 3 del; 1 mod
  Patch: https://git.openjdk.org/lilliput-jdk17u/pull/23.diff
  Fetch: git fetch https://git.openjdk.org/lilliput-jdk17u.git pull/23/head:pull/23

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


More information about the lilliput-dev mailing list