[lworld] Integrated: 8368988: [lworld] Rename classes in valhalla/inlinetypes to be distinct

Marc Chevalier mchevalier at openjdk.org
Tue Oct 21 10:23:55 UTC 2025


On Fri, 17 Oct 2025 11:27:47 GMT, Marc Chevalier <mchevalier at openjdk.org> wrote:

> Various cleanup in test package `compiler.valhalla.inlinetypes`:
> - rename duplicated classes so that IDEs are nice, resolve correctly. New names are a bit... arbitrary. I'm happy to rename name if anyone has opinions. When writing new tests, let's prefer nested classes if we can.
> - add `package compiler.valhalla.inlinetypes;` where it was missing. IDEs like that, and I vaguely remember we should.
> - fix flags to take the new path into account, new package, or class renaming. Please, try to pay attention to that: I've did multiple passes to catch'em all, but this is easy to miss.
> - remove some useless `import`.
> - some few typos in comments.
> - maybe more?
> 
> Thanks,
> Marc

This pull request has now been integrated.

Changeset: e0aaf290
Author:    Marc Chevalier <mchevalier at openjdk.org>
URL:       https://git.openjdk.org/valhalla/commit/e0aaf29064ed67c240f5275e9b3d324ce5550382
Stats:     1587 lines in 54 files changed: 40 ins; 53 del; 1494 mod

8368988: [lworld] Rename classes in valhalla/inlinetypes to be distinct

Reviewed-by: thartmann

-------------

PR: https://git.openjdk.org/valhalla/pull/1683


More information about the valhalla-dev mailing list