On Thu, 17 Feb 2022 15:11:11 GMT, Claes Redestad <redes...@openjdk.org> wrote:
> Splitting out these micro changes from #7231 > > - Clean up and simplify setup and code > - Add variants with different inputs with varying lengths and encoding > weights, but also relevant mixes of each so that we both cover interesting > corner cases but also verify that performance behave when there's a multitude > of input shapes. Both simple and mixed variants are interesting diagnostic > tools. > - Drop all charsets from the default run configuration except UTF-8. > Motivation: Defaults should give good coverage but try to keep runtimes at > bay. Additionally if the charset tested can't encode the higher codepoints > used in these micros the results might be misleading. If you for example test > using ISO-8859-1 the UTF-16 bytes in StringDecode.decodeUTF16 will have all > been replaced by `?`, so the test is effectively the same as testing > ASCII-only. This pull request has now been integrated. Changeset: d4d12ad1 Author: Claes Redestad <redes...@openjdk.org> URL: https://git.openjdk.java.net/jdk/commit/d4d12ad1d92ae6d353ce9e16b1b72e6416874cd6 Stats: 358 lines in 2 files changed: 218 ins; 71 del; 69 mod 8282047: Enhance StringDecode/Encode microbenchmarks Reviewed-by: bchristi ------------- PR: https://git.openjdk.java.net/jdk/pull/7516