[lworld] Integrated: 8304168: [lworld] CDS tests fail with --enable-preview patched value classes
Ioi Lam
iklam at openjdk.org
Sun Mar 31 01:37:44 UTC 2024
On Tue, 26 Mar 2024 15:51:48 GMT, Ioi Lam <iklam at openjdk.org> wrote:
> [1] Allow `--enable-preview -XX:+EnableValhalla` to be specified when dumping a CDS archive. This allows only the patch modules that are used for the value classes. The user is still forbidden from specifying --patch-module when dumping the CDS archive.
>
> [2] If a CDS archive was created with `--enable-preview -XX:+EnableValhalla`, then it must also be specified at run time, or else the CDS archive will be ignored.
>
> [3] Fixed a few bugs with value types. E.g., some metadata objects need to be placed in R/W region.
>
> [4] Tests with all CDS tests and tiers 1-4.
>
> [5] Modify Image.gmk to be able to generate default CDS archives for `--enable-preview -XX:+EnableValhalla`, but this is not yet enabled as some non-CDS tests will fail. See [JDK-8329108](https://bugs.openjdk.org/browse/JDK-8329108)
This pull request has now been integrated.
Changeset: 7ea9a235
Author: Ioi Lam <iklam at openjdk.org>
URL: https://git.openjdk.org/valhalla/commit/7ea9a235e8bbc7787d55d6f1723bee15e711b7ad
Stats: 252 lines in 25 files changed: 149 ins; 23 del; 80 mod
8304168: [lworld] CDS tests fail with --enable-preview patched value classes
Reviewed-by: mchung, fparain
-------------
PR: https://git.openjdk.org/valhalla/pull/1063
More information about the valhalla-dev
mailing list