[lworld] Integrated: 8372686: [lworld] runtime/cds/serviceability/ReplaceCriticalClasses.java fails with --enable-preview
Coleen Phillimore
coleenp at openjdk.org
Thu Dec 18 18:30:50 UTC 2025
On Thu, 18 Dec 2025 13:11:05 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:
> 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.
This pull request has now been integrated.
Changeset: 1e6e72e2
Author: Coleen Phillimore <coleenp at openjdk.org>
URL: https://git.openjdk.org/valhalla/commit/1e6e72e246608d872d7733559da7ba29ed10f69a
Stats: 6 lines in 2 files changed: 0 ins; 2 del; 4 mod
8372686: [lworld] runtime/cds/serviceability/ReplaceCriticalClasses.java fails with --enable-preview
Reviewed-by: iklam
-------------
PR: https://git.openjdk.org/valhalla/pull/1827
More information about the valhalla-dev
mailing list