RFR: 8333833: UUID toString removes the use of ByteArrayLittleEndian [v2]
Glavo
duke at openjdk.org
Sat Jun 8 14:51:11 UTC 2024
On Sat, 8 Jun 2024 06:40:39 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:
>
> add comments
I think we don't need to change them back everywhere, but only need to rewrite `ByteArrayLittleEndian` and `ByteArray` so that they no longer use `VarHandle`.
Maybe I should rewrite https://github.com/openjdk/jdk/pull/14636 without using `Unsafe`, so more people might agree with it.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/19610#issuecomment-2156063742
More information about the core-libs-dev
mailing list