On Sat, 1 Jul 2023 03:06:31 GMT, Chen Liang <li...@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

Reply via email to