RFR: 8311207: Optimization for j.u.UUID.toString

温绍锦 duke at openjdk.org
Sat Jul 1 03:46:54 UTC 2023


On Sat, 1 Jul 2023 03:06:31 GMT, Chen Liang <liach at openjdk.org> wrote:

>> ![image](https://github.com/openjdk/jdk/assets/1166785/38c75145-b685-4c4e-a53a-97f8c44778e3)
>> 
>> it's enabled
>
>> > @wenshao I have made my suggestions into a patch for you: [wenshao#1](https://github.com/wenshao/jdk/pull/1) Feel free to review.
>> 
>> your version performance is a bit worse, if i can't find the reason, I will restore the previous version
> 
> Can you post the benchmark results? And do you have like a baseline for the benchmarks, as there may be other factors that affect performance from run to run?
> 
> Also, my fault for actions: can you go to your actions tab and enable actions for your fork like this?
> <img width="1267" alt="image" src="https://github.com/openjdk/jdk/assets/7806504/d049d88e-e4fc-4d35-b2a4-87fbfe82dbd5">

@liach 

## MacBookPro M1 Pro
* Update 02
current version [76dba964](https://git.openjdk.org/jdk/pull/14745/files/76dba9649a530a2f21574f3dff1aa55a6ebe8888)

Benchmark           (size)   Mode  Cnt    Score   Error   Units 
UUIDBench.toString   20000  thrpt   15  104.262 ± 2.199  ops/us


* Update 01
your version [425559f6](https://git.openjdk.org/jdk/pull/14745/files/425559f65a3e5f72323abdadbbf9df426e196d9b)


Benchmark           (size)   Mode  Cnt   Score   Error   Units
UUIDBench.toString   20000  thrpt   15  81.622 ± 0.194  ops/us

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

PR Comment: https://git.openjdk.org/jdk/pull/14745#issuecomment-1615435719


More information about the core-libs-dev mailing list