On Tue, 10 Mar 2026 16:49:15 GMT, Paul Hübner <[email protected]> wrote:

>> Frederic Parain has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Fix generation of short values
>
> test/hotspot/jtreg/runtime/valhalla/inlinetypes/field_layout/ValueClassGenerator.java
>  line 497:
> 
>> 495:     }
>> 496: 
>> 497:     void printStatistics(ArrayList<? extends ValueClassDesc> list) {
> 
> This whole method could probably be simplified with a mapreduce-style 
> stream/collect but it will get the job done.

Initially, it was a debug code that I kept because it can give a quick insight 
if the distributions used to generate the classes behave as expected or not. 
Again, not a critical part of the code.

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

PR Review Comment: 
https://git.openjdk.org/valhalla/pull/2202#discussion_r2918313039

Reply via email to