Re: RFR: 8339699: Optimize DataOutputStream writeUTF [v6]

2024-10-04 Thread Brian Burkhalter
On Fri, 20 Sep 2024 05:09:19 GMT, Shaojin Wen wrote: >> PR #20772 introduced an optimization for writeUTF, which can also be used in >> DataOutputStream::writeUTF. > > Shaojin Wen has updated the pull request incrementally with one additional > commit since the last revision: > > bug fix for

Re: RFR: 8339699: Optimize DataOutputStream writeUTF [v6]

2024-09-20 Thread Chen Liang
On Fri, 20 Sep 2024 05:09:19 GMT, Shaojin Wen wrote: >> PR #20772 introduced an optimization for writeUTF, which can also be used in >> DataOutputStream::writeUTF. > > Shaojin Wen has updated the pull request incrementally with one additional > commit since the last revision: > > bug fix for

Re: RFR: 8339699: Optimize DataOutputStream writeUTF [v5]

2024-09-19 Thread Shaojin Wen
On Thu, 19 Sep 2024 19:00:45 GMT, Chen Liang wrote: >> Shaojin Wen has updated the pull request incrementally with one additional >> commit since the last revision: >> >> suggestion from @liach > > src/java.base/share/classes/java/io/ObjectOutputStream.java line 2047: > >> 2045:

Re: RFR: 8339699: Optimize DataOutputStream writeUTF [v6]

2024-09-19 Thread Shaojin Wen
> PR #20772 introduced an optimization for writeUTF, which can also be used in > DataOutputStream::writeUTF. Shaojin Wen has updated the pull request incrementally with one additional commit since the last revision: bug fix for write long utf - Changes: - all: https://git.open

Re: RFR: 8339699: Optimize DataOutputStream writeUTF [v5]

2024-09-19 Thread Chen Liang
On Mon, 16 Sep 2024 14:23:52 GMT, Shaojin Wen wrote: >> PR #20772 introduced an optimization for writeUTF, which can also be used in >> DataOutputStream::writeUTF. > > Shaojin Wen has updated the pull request incrementally with one additional > commit since the last revision: > > suggestion

Re: RFR: 8339699: Optimize DataOutputStream writeUTF [v5]

2024-09-16 Thread Chen Liang
On Mon, 16 Sep 2024 14:23:52 GMT, Shaojin Wen wrote: >> PR #20772 introduced an optimization for writeUTF, which can also be used in >> DataOutputStream::writeUTF. > > Shaojin Wen has updated the pull request incrementally with one additional > commit since the last revision: > > suggestion

Re: RFR: 8339699: Optimize DataOutputStream writeUTF [v5]

2024-09-16 Thread Shaojin Wen
> PR #20772 introduced an optimization for writeUTF, which can also be used in > DataOutputStream::writeUTF. Shaojin Wen has updated the pull request incrementally with one additional commit since the last revision: suggestion from @liach - Changes: - all: https://git.openjdk.

Re: RFR: 8339699: Optimize DataOutputStream writeUTF [v4]

2024-09-16 Thread Chen Liang
On Fri, 13 Sep 2024 17:38:56 GMT, Shaojin Wen wrote: >> PR #20772 introduced an optimization for writeUTF, which can also be used in >> DataOutputStream::writeUTF. > > Shaojin Wen has updated the pull request incrementally with two additional > commits since the last revision: > > - rename JD

Re: RFR: 8339699: Optimize DataOutputStream writeUTF [v3]

2024-09-13 Thread Shaojin Wen
On Fri, 13 Sep 2024 17:05:11 GMT, Chen Liang wrote: >> Shaojin Wen has updated the pull request incrementally with one additional >> commit since the last revision: >> >> reduce JDKUTF#utflen codeSize > > src/java.base/share/classes/jdk/internal/util/JDKUTF.java line 37: > >> 35: public abst

Re: RFR: 8339699: Optimize DataOutputStream writeUTF [v4]

2024-09-13 Thread Shaojin Wen
> PR #20772 introduced an optimization for writeUTF, which can also be used in > DataOutputStream::writeUTF. Shaojin Wen has updated the pull request incrementally with two additional commits since the last revision: - rename JDKUTF to ModifiedUtf - suggestion from @liach - Chan

Re: RFR: 8339699: Optimize DataOutputStream writeUTF [v3]

2024-09-13 Thread Chen Liang
On Sat, 7 Sep 2024 08:51:25 GMT, Shaojin Wen wrote: >> PR #20772 introduced an optimization for writeUTF, which can also be used in >> DataOutputStream::writeUTF. > > Shaojin Wen has updated the pull request incrementally with one additional > commit since the last revision: > > reduce JDKUT

Re: RFR: 8339699: Optimize DataOutputStream writeUTF [v3]

2024-09-07 Thread Shaojin Wen
On Sat, 7 Sep 2024 14:04:55 GMT, ExE Boss wrote: >> Shaojin Wen has updated the pull request incrementally with one additional >> commit since the last revision: >> >> reduce JDKUTF#utflen codeSize > > src/java.base/share/classes/java/io/DataOutputStream.java line 382: > >> 380: Byte

Re: RFR: 8339699: Optimize DataOutputStream writeUTF [v3]

2024-09-07 Thread ExE Boss
On Sat, 7 Sep 2024 08:51:25 GMT, Shaojin Wen wrote: >> PR #20772 introduced an optimization for writeUTF, which can also be used in >> DataOutputStream::writeUTF. > > Shaojin Wen has updated the pull request incrementally with one additional > commit since the last revision: > > reduce JDKUT

Re: RFR: 8339699: Optimize DataOutputStream writeUTF [v3]

2024-09-07 Thread Shaojin Wen
> PR #20772 introduced an optimization for writeUTF, which can also be used in > DataOutputStream::writeUTF. Shaojin Wen has updated the pull request incrementally with one additional commit since the last revision: reduce JDKUTF#utflen codeSize - Changes: - all: https://git.o

Re: RFR: 8339699: Optimize DataOutputStream writeUTF [v2]

2024-09-07 Thread Shaojin Wen
> PR #20772 introduced an optimization for writeUTF, which can also be used in > DataOutputStream::writeUTF. Shaojin Wen has updated the pull request incrementally with one additional commit since the last revision: code style - Changes: - all: https://git.openjdk.org/jdk/pull

Re: RFR: 8339699: Optimize DataOutputStream writeUTF

2024-09-07 Thread Shaojin Wen
On Fri, 6 Sep 2024 09:58:58 GMT, Shaojin Wen wrote: > PR #20772 introduced an optimization for writeUTF, which can also be used in > DataOutputStream::writeUTF. Below are the performance numbers running on a MacBook M1 Pro. Both DataOutputStreamBench::writeUTF and ObjectOutputStream::writeUTF