RFR: JDK-8310502 : Optimization for j.l.Long.fastUUID() [v19]
温绍锦
duke at openjdk.org
Sun Jun 25 07:08:19 UTC 2023
On Fri, 23 Jun 2023 04:52:16 GMT, 温绍锦 <duke at openjdk.org> wrote:
>> 温绍锦 has updated the pull request incrementally with one additional commit since the last revision:
>>
>> remove unused import
>
> Latest benchmark results (commit id [0d34655](https://github.com/openjdk/jdk/pull/14578/commits/0d346558e00cfab899e435acd0eb83d39d5b7169) ):
>
> * [aliyun_c8i.xlarge](https://help.aliyun.com/document_detail/25378.html#c8i) (+32.17% )
> cpu : intel xeon sapphire rapids (x64)
> ``` diff
> Benchmark (size) Mode Cnt Score Error Units
> -UUIDBench.toString 20000 thrpt 15 62.457 ± 0.226 ops/us
> +UUIDBench.toString 20000 thrpt 15 82.550 ± 0.704 ops/us
>
>
> * [aliyun_c8a.xlarge](https://help.aliyun.com/document_detail/25378.html#c8a) (+46.88%)
> cpu : amd epc genoa (x64)
> ``` diff
> Benchmark (size) Mode Cnt Score Error Units
> -UUIDBench.toString 20000 thrpt 15 65.673 ± 0.110 ops/us
> +UUIDBench.toString 20000 thrpt 15 96.461 ± 0.293 ops/us
>
>
> * [aliyun_c8y.xlarge](https://help.aliyun.com/document_detail/25378.html#c8y) (+57.97%)
> cpu : aliyun yitian 710 (aarch64)
> ``` diff
> Benchmark (size) Mode Cnt Score Error Units
> -UUIDBench.toString 20000 thrpt 15 32.198 ± 0.907 ops/us
> +UUIDBench.toString 20000 thrpt 15 50.864 ± 1.305 ops/us
>
>
> * aws_c5.xlarge (+49.42% )
> aws x64
> ``` diff
> Benchmark (size) Mode Cnt Score Error Units
> -UUIDBench.toString 20000 thrpt 15 46.003 ± 0.291 ops/us
> +UUIDBench.toString 20000 thrpt 15 68.741 ± 0.199 ops/us
>
>
> * MacBook Pro M1 Pro (+201.14%)
> ``` diff
> Benchmark (size) Mode Cnt Score Error Units
> -UUIDBench.toString 20000 thrpt 15 36.875 ± 0.902 ops/us
> +UUIDBench.toString 20000 thrpt 15 111.049 ± 0.783 ops/us
>
>
> * [Orange Pi 5 Plus](http://www.orangepi.org/html/hardWare/computerAndMicrocontrollers/details/Orange-Pi-5-plus.html) (+64.97%)
> cpu : RK3588 (aarch64)
> ``` diff
> Benchmark (size) Mode Cnt Score Error Units
> -UUIDBench.toString 20000 thrpt 15 21.691 ± 0.204 ops/us
> +UUIDBench.toString 20000 thrpt 15 35.785 ± 0.447 ops/us
> @wenshao The patch has changed 19 times in the last 3 days, is it ready for review now?
@AlanBateman
it's ready for review,. thank you.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/14578#issuecomment-1605903338
More information about the core-libs-dev
mailing list