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