On Tue, 1 Jul 2025 17:02:27 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:
> 
>   Adding comment about maintainability.

Oddly while the status at the bottom of the conversation says it has "2 
approvals", the tickbox:


Change must be properly reviewed (2 reviews required, with at least 1 Reviewer, 
1 Author


is not checked. I can try to integrate, but it might not work.

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

PR Comment: https://git.openjdk.org/jdk/pull/26044#issuecomment-3075753460

Reply via email to