[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