On Mon, 17 Nov 2025 16:09:42 GMT, Matias Saavedra Silva <[email protected]>
wrote:
> This patch fixes a bug where the VM tries to pre-register migrated value
> classes regardless of whether or not the class loader data already exists.
> Verified with tier 1-5 tests.
src/hotspot/share/classfile/systemDictionary.cpp line 216:
> 214: bool created = false;
> 215: ClassLoaderData* cld =
> ClassLoaderDataGraph::find_or_create(class_loader, created);
> 216: if (created && Arguments::enable_preview() && EnableValhalla) {
I'd like to get rid of the `EnableValhalla` while we are at it,
`--enable-preview` implies `EnableValhalla`, plus this would mean one less
place to remove the flag later.
-------------
PR Review Comment:
https://git.openjdk.org/valhalla/pull/1741#discussion_r2537300237
