[lworld] RFR: 8371826: [lworld] Migrated value classes pre-registered unconditionally

[Paul Hübner]

On Mon, 17 Nov 2025 16:09:42 GMT, Matias Saavedra Silva <matsaave at openjdk.org> 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