On Mon, 30 Jun 2025 15:08:25 GMT, David Beaumont <d...@openjdk.org> wrote:

>> Initial benchmark to capture at least some comparative measures of 
>> ImageReader performance.
>> 
>> Current results on my laptop:
>> 
>> Benchmark                                         Mode  Cnt   Score    Error 
>>  Units
>> NewImageBenchmark.warmCache_CountAllNodes         avgt    5   0.785 ±  0.140 
>>  ms/op
>> NewImageBenchmark.coldStart_CountOnly               ss    5  34.051 ± 17.662 
>>  ms/op
>> NewImageBenchmark.coldStart_InitAndCount            ss    5  31.006 ±  9.674 
>>  ms/op
>> NewImageBenchmark.coldStart_LoadJavacInitClasses    ss    5   3.752 ±  6.873 
>>  ms/op
>> 
>> The upcoming changes to ImageReader should show significant improvements to 
>> these results.
>
> David Beaumont has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Some tidying up relating to the state objects.

test/micro/org/openjdk/bench/jdk/internal/jrtfs/ImageReaderBenchmark.java line 
230:

> 228:     // Created by running "java -verbose:class", throwing away anonymous 
> inner
> 229:     // classes and anything without a reliable name, and grouping by the 
> stated
> 230:     // source. It's not perfect, but it's representative.

I don't think this is maintainable. How useful (or not) is this benchmark if 
the names of all the internal classes (that will change from release to 
release) are dropped from this?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/26044#discussion_r2176636106

Reply via email to