RFR: 8361076: Add benchmark for ImageReader in preparation for Valhalla changes [v2]

David Beaumont duke at openjdk.org
Mon Jun 30 13:21:06 UTC 2025


On Mon, 30 Jun 2025 13:18:19 GMT, David Beaumont <duke at 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:
> 
>   Renaming class.

test/micro/org/openjdk/bench/jdk/internal/jrtfs/NewImageBenchmark.java line 71:

> 69:     }
> 70: 
> 71:     /// NOT a @State since that causes setUp()/tearDown() to be shared, but we

This comment might have been before I applied `Scope.Benchmark` to the states below. I'll double check if it's still an issue.

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

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


More information about the build-dev mailing list