On Wed, 12 Apr 2023 17:39:16 GMT, Roger Riggs <rri...@openjdk.org> wrote:

>> Sergey Tsypanov has updated the pull request with a new target base due to a 
>> merge or a rebase. The incremental webrev excludes the unrelated changes 
>> brought in by the merge/rebase. The pull request contains six additional 
>> commits since the last revision:
>> 
>>  - Merge branch 'master' into dtfb
>>  - 8300818: Add benchmark
>>  - 8300818: Add benchmark
>>  - Revert irrelevant change
>>  - Merge branch 'master' into dtfb
>>  - Improve padding of DateTimeFormatter
>
> test/micro/org/openjdk/bench/java/time/format/DateTimeFormatterBench.java 
> line 148:
> 
>> 146:         return YEAR_FORMATTER.format(now);
>> 147:     }
>> 148: 
> 
> There benchmarks are run 4 time each because of the @Param that are defined 
> for the pre-existing tests.
> I don't know if jmh can be annotated to ignore the params; but to avoid 
> wasting resources create a separate jmh source file for thees new tests.

Done

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

PR Review Comment: https://git.openjdk.org/jdk/pull/12131#discussion_r1167835249

Reply via email to