[lworld] RFR: 8372686: [lworld] runtime/cds/serviceability/ReplaceCriticalClasses.java fails with --enable-preview
Coleen Phillimore
coleenp at openjdk.org
Thu Dec 18 13:16:56 UTC 2025
This replaces Locale with Calendar and removes BaseLocale to test loading with ClassFileLoadHook, like my previous pull request. Also removes the test from the ProblemList-enable-valhalla.txt file.
BaseLocale is also loaded and I thought it was the migrated classes that caused early loading for Locale, but it was really the module patching that we used to do for --enable-preview. See call stack in bug report.
Tested with the test on windows, where it failed.
-------------
Commit messages:
- 8372686: [lworld] runtime/cds/serviceability/ReplaceCriticalClasses.java fails with --enable-preview
Changes: https://git.openjdk.org/valhalla/pull/1827/files
Webrev: https://webrevs.openjdk.org/?repo=valhalla&pr=1827&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8372686
Stats: 6 lines in 2 files changed: 0 ins; 2 del; 4 mod
Patch: https://git.openjdk.org/valhalla/pull/1827.diff
Fetch: git fetch https://git.openjdk.org/valhalla.git pull/1827/head:pull/1827
PR: https://git.openjdk.org/valhalla/pull/1827
More information about the valhalla-dev
mailing list