RFR: 8333833: UUID toString removes the use of ByteArrayLittleEndian [v3]

Shaojin Wen duke at openjdk.org
Sun Jun 9 05:47:17 UTC 2024


On Sat, 8 Jun 2024 23:30:38 GMT, Shaojin Wen <duke at openjdk.org> wrote:

>> After PR #16245, C2 optimizes stores into primitive arrays by combining values ​​into larger stores. In the UUID.toString method, ByteArrayLittleEndian can be removed, making the code more elegant and faster.
>
> Shaojin Wen has updated the pull request incrementally with one additional commit since the last revision:
> 
>   change method name, putHex -> putHex4, and fix comments

> > You are right, ByteArray and ByteArrayLittleEndian have good performance after removing Unsafe. This is similar to the previous version of java.io.Bits
> 
> Do you have evidence that `VarHandle` affects startup time? If there is good evidence to support this, I would go ahead and rewrite #14636.

@cl4es has fixed startup regression issues, such as this https://github.com/openjdk/jdk/pull/15836

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

PR Comment: https://git.openjdk.org/jdk/pull/19610#issuecomment-2156332130


More information about the core-libs-dev mailing list