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

2024-06-09 Thread Chen Liang
On Sat, 8 Jun 2024 23:30:38 GMT, Shaojin Wen 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 u

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

2024-06-08 Thread Shaojin Wen
On Sat, 8 Jun 2024 23:30:38 GMT, Shaojin Wen 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 u

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

2024-06-08 Thread Glavo
On Sun, 9 Jun 2024 01:16:49 GMT, Shaojin Wen wrote: > 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

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

2024-06-08 Thread Shaojin Wen
On Sat, 8 Jun 2024 23:30:38 GMT, Shaojin Wen 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 u

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

2024-06-08 Thread Shaojin Wen
> 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 commi