RFR: 8310837: Use ByteArrayLittleEndian in java.util.zip [v8]
Chen Liang
liach at openjdk.org
Thu May 9 12:03:59 UTC 2024
On Wed, 28 Feb 2024 14:21:16 GMT, Glavo <duke at openjdk.org> wrote:
>> Using `ByteArrayLittleEndian` is simpler and faster.
>>
>> `make test TEST="micro:java.util.zip.ZipFileOpen"`:
>>
>>
>> Benchmark (size) Mode Cnt Score Error Units
>> - ZipFileOpen.openCloseZipFile 512 avgt 15 39052.832 ± 107.496 ns/op
>> + ZipFileOpen.openCloseZipFile 512 avgt 15 36275.539 ± 663.193 ns/op
>> - ZipFileOpen.openCloseZipFile 1024 avgt 15 77106.494 ± 4159.300 ns/op
>> + ZipFileOpen.openCloseZipFile 1024 avgt 15 71955.013 ± 2296.050 ns/op
>
> Glavo 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 eight additional commits since the last revision:
>
> - Merge branch 'openjdk:master' into zip-utils
> - Add tests
> - Merge branch 'openjdk:master' into zip-utils
> - Merge branch 'openjdk:master' into zip-utils
> - Merge branch 'openjdk:master' into zip-utils
> - Merge branch 'openjdk:master' into zip-utils
> - Merge branch 'openjdk:master' into zip-utils
> - use ByteArrayLittleEndian in ZipUtils
@cl4es Can you take a look at this performance improvement patch?
-------------
PR Comment: https://git.openjdk.org/jdk/pull/14632#issuecomment-2102527844
More information about the core-libs-dev
mailing list