On Wed, 22 Apr 2026 08:53:15 GMT, Paul Hübner <[email protected]> wrote:
>> Currently JITTester only generates common, instance classes. This PR >> suggests a value classes support for the tool. >> >> Specifically, it: >> - adds a ValueKlass as an AST node carrier for the new type >> - moves most of KlassFactory internals into generic AbstractKlassFactory >> - adds smaller concrete KlassFactory and ValueKlassFactory. >> - makes sure that we don't generate mutable fields in the value classes >> - blocks synchronized usage on value-class methods >> - raises probability to be abstract for the value classes - to check deeper >> hierarchies. >> - adds a parameter governing the balance between instance and value classes. >> >> --------- >> - [x] I confirm that I make this contribution in accordance with the >> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai). > > test/hotspot/jtreg/testlibrary/jittester/src/jdk/test/lib/jittester/classes/ValueKlass.java > line 34: > >> 32: >> 33: /** >> 34: * A simple POJO representing Value Class in the AST. > > Would there be value (pun not intended) in generating value records at some > point (in the future)? Yes, of course. The more language structures we cover, the better, I guess. ------------- PR Review Comment: https://git.openjdk.org/valhalla/pull/2346#discussion_r3125444667
