Re: RFR: 8341141: Optimize DirectCodeBuilder [v24]

2024-10-08 Thread Chen Liang
On Tue, 8 Oct 2024 22:51:46 GMT, Shaojin Wen wrote: >> Indeed, the opcode here is pointless if we have a pool adaption; in that >> case we should just use `ldc(LoadableConstantEntry)`. > > Here in writeLoadConstant, some branches have curly braces, and some don't. > The style is inconsistent. S

Re: RFR: 8341141: Optimize DirectCodeBuilder [v24]

2024-10-08 Thread Shaojin Wen
On Tue, 8 Oct 2024 22:32:24 GMT, Chen Liang wrote: >> src/java.base/share/classes/jdk/internal/classfile/impl/DirectCodeBuilder.java >> line 706: >> >>> 704: public void writeLoadConstant(Opcode opcode, LoadableConstantEntry >>> value) { >>> 705: // Make sure Long and Double have L

Re: RFR: 8341141: Optimize DirectCodeBuilder [v25]

2024-10-08 Thread Claes Redestad
On Tue, 8 Oct 2024 22:25:03 GMT, Shaojin Wen wrote: > Are you talking about merging multiple writeU2 in places like > AbstractAttributeMapper? I am planning to make a new PR to complete these > works, because this PR has too many changes. Good. I would have suggested the same. -

Re: RFR: 8341141: Optimize DirectCodeBuilder [v24]

2024-10-08 Thread Chen Liang
On Tue, 8 Oct 2024 09:34:42 GMT, Claes Redestad 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/jdk/internal/classfile/impl/DirectCodeBuilder.java > line 706:

Re: RFR: 8341141: Optimize DirectCodeBuilder [v26]

2024-10-08 Thread Chen Liang
On Tue, 8 Oct 2024 22:22:37 GMT, Shaojin Wen wrote: >> Some DirectCodeBuilder related optimizations to improve startup and running >> performance: >> 1. Merge calls, merge writeU1 and writeU2 into writeU3 >> 2. Merge calls, merge writeU1 and writeIndex operations >> 3. Directly use writeU1 inste

Re: RFR: 8341141: Optimize DirectCodeBuilder [v25]

2024-10-08 Thread Shaojin Wen
On Tue, 8 Oct 2024 22:15:23 GMT, Claes Redestad wrote: > LGTM. > > I spotted places outside of the DirectCodeBuilder paths that could benefit > from using these new coalescing writers, but write-only cases is a good focus > point for now. Are you talking about merging multiple writeU2 in plac

Re: RFR: 8341141: Optimize DirectCodeBuilder [v26]

2024-10-08 Thread Shaojin Wen
> Some DirectCodeBuilder related optimizations to improve startup and running > performance: > 1. Merge calls, merge writeU1 and writeU2 into writeU3 > 2. Merge calls, merge writeU1 and writeIndex operations > 3. Directly use writeU1 instead of writeBytecode > 4. Rewrite the implementation of load

Re: RFR: 8341141: Optimize DirectCodeBuilder [v25]

2024-10-08 Thread Claes Redestad
On Tue, 8 Oct 2024 09:57:43 GMT, Shaojin Wen wrote: >> Some DirectCodeBuilder related optimizations to improve startup and running >> performance: >> 1. Merge calls, merge writeU1 and writeU2 into writeU3 >> 2. Merge calls, merge writeU1 and writeIndex operations >> 3. Directly use writeU1 inste

Re: RFR: 8341141: Optimize DirectCodeBuilder [v24]

2024-10-08 Thread Claes Redestad
On Mon, 7 Oct 2024 22:54:24 GMT, Shaojin Wen wrote: >> Some DirectCodeBuilder related optimizations to improve startup and running >> performance: >> 1. Merge calls, merge writeU1 and writeU2 into writeU3 >> 2. Merge calls, merge writeU1 and writeIndex operations >> 3. Directly use writeU1 inste

Re: RFR: 8341141: Optimize DirectCodeBuilder [v25]

2024-10-08 Thread Chen Liang
On Tue, 8 Oct 2024 09:57:43 GMT, Shaojin Wen wrote: >> Some DirectCodeBuilder related optimizations to improve startup and running >> performance: >> 1. Merge calls, merge writeU1 and writeU2 into writeU3 >> 2. Merge calls, merge writeU1 and writeIndex operations >> 3. Directly use writeU1 inste

Re: RFR: 8341141: Optimize DirectCodeBuilder [v25]

2024-10-08 Thread Shaojin Wen
> Some DirectCodeBuilder related optimizations to improve startup and running > performance: > 1. Merge calls, merge writeU1 and writeU2 into writeU3 > 2. Merge calls, merge writeU1 and writeIndex operations > 3. Directly use writeU1 instead of writeBytecode > 4. Rewrite the implementation of load

Re: RFR: 8341141: Optimize DirectCodeBuilder [v24]

2024-10-07 Thread Shaojin Wen
> Some DirectCodeBuilder related optimizations to improve startup and running > performance: > 1. Merge calls, merge writeU1 and writeU2 into writeU3 > 2. Merge calls, merge writeU1 and writeIndex operations > 3. Directly use writeU1 instead of writeBytecode > 4. Rewrite the implementation of load

Re: RFR: 8341141: Optimize DirectCodeBuilder [v23]

2024-10-07 Thread Claes Redestad
On Mon, 7 Oct 2024 20:03:29 GMT, Chen Liang wrote: >> Shaojin Wen has updated the pull request incrementally with one additional >> commit since the last revision: >> >> fix merge error > > src/java.base/share/classes/jdk/internal/classfile/impl/DirectCodeBuilder.java > line 989: > >> 987:

Re: RFR: 8341141: Optimize DirectCodeBuilder [v23]

2024-10-07 Thread Chen Liang
On Sat, 5 Oct 2024 16:33:50 GMT, Shaojin Wen wrote: >> Some DirectCodeBuilder related optimizations to improve startup and running >> performance: >> 1. Merge calls, merge writeU1 and writeU2 into writeU3 >> 2. Merge calls, merge writeU1 and writeIndex operations >> 3. Directly use writeU1 inste

Re: RFR: 8341141: Optimize DirectCodeBuilder [v23]

2024-10-07 Thread Claes Redestad
On Sat, 5 Oct 2024 16:33:50 GMT, Shaojin Wen wrote: >> Some DirectCodeBuilder related optimizations to improve startup and running >> performance: >> 1. Merge calls, merge writeU1 and writeU2 into writeU3 >> 2. Merge calls, merge writeU1 and writeIndex operations >> 3. Directly use writeU1 inste

Re: RFR: 8341141: Optimize DirectCodeBuilder [v23]

2024-10-05 Thread Shaojin Wen
> Some DirectCodeBuilder related optimizations to improve startup and running > performance: > 1. Merge calls, merge writeU1 and writeU2 into writeU3 > 2. Merge calls, merge writeU1 and writeIndex operations > 3. Directly use writeU1 instead of writeBytecode > 4. Rewrite the implementation of load

Re: RFR: 8341141: Optimize DirectCodeBuilder [v22]

2024-10-05 Thread Shaojin Wen
> Some DirectCodeBuilder related optimizations to improve startup and running > performance: > 1. Merge calls, merge writeU1 and writeU2 into writeU3 > 2. Merge calls, merge writeU1 and writeIndex operations > 3. Directly use writeU1 instead of writeBytecode > 4. Rewrite the implementation of load

Re: RFR: 8341141: Optimize DirectCodeBuilder [v21]

2024-10-05 Thread Shaojin Wen
> Some DirectCodeBuilder related optimizations to improve startup and running > performance: > 1. Merge calls, merge writeU1 and writeU2 into writeU3 > 2. Merge calls, merge writeU1 and writeIndex operations > 3. Directly use writeU1 instead of writeBytecode > 4. Rewrite the implementation of load

Re: RFR: 8341141: Optimize DirectCodeBuilder [v20]

2024-10-03 Thread Shaojin Wen
> Some DirectCodeBuilder related optimizations to improve startup and running > performance: > 1. Merge calls, merge writeU1 and writeU2 into writeU3 > 2. Merge calls, merge writeU1 and writeIndex operations > 3. Directly use writeU1 instead of writeBytecode > 4. Rewrite the implementation of load

Re: RFR: 8341141: Optimize DirectCodeBuilder [v19]

2024-10-03 Thread Chen Liang
On Wed, 2 Oct 2024 04:32:10 GMT, Shaojin Wen wrote: >> Some DirectCodeBuilder related optimizations to improve startup and running >> performance: >> 1. Merge calls, merge writeU1 and writeU2 into writeU3 >> 2. Merge calls, merge writeU1 and writeIndex operations >> 3. Directly use writeU1 inste

Re: RFR: 8341141: Optimize DirectCodeBuilder [v19]

2024-10-01 Thread Shaojin Wen
> Some DirectCodeBuilder related optimizations to improve startup and running > performance: > 1. Merge calls, merge writeU1 and writeU2 into writeU3 > 2. Merge calls, merge writeU1 and writeIndex operations > 3. Directly use writeU1 instead of writeBytecode > 4. Rewrite the implementation of load

Re: RFR: 8341141: Optimize DirectCodeBuilder [v18]

2024-10-01 Thread Shaojin Wen
> Some DirectCodeBuilder related optimizations to improve startup and running > performance: > 1. Merge calls, merge writeU1 and writeU2 into writeU3 > 2. Merge calls, merge writeU1 and writeIndex operations > 3. Directly use writeU1 instead of writeBytecode > 4. Rewrite the implementation of load

Re: RFR: 8341141: Optimize DirectCodeBuilder [v17]

2024-10-01 Thread Shaojin Wen
> Some DirectCodeBuilder related optimizations to improve startup and running > performance: > 1. Merge calls, merge writeU1 and writeU2 into writeU3 > 2. Merge calls, merge writeU1 and writeIndex operations > 3. Directly use writeU1 instead of writeBytecode > 4. Rewrite the implementation of load

Re: RFR: 8341141: Optimize DirectCodeBuilder [v16]

2024-10-01 Thread Shaojin Wen
> Some DirectCodeBuilder related optimizations to improve startup and running > performance: > 1. Merge calls, merge writeU1 and writeU2 into writeU3 > 2. Merge calls, merge writeU1 and writeIndex operations > 3. Directly use writeU1 instead of writeBytecode > 4. Rewrite the implementation of load

Re: RFR: 8341141: Optimize DirectCodeBuilder [v15]

2024-10-01 Thread Shaojin Wen
> Some DirectCodeBuilder related optimizations to improve startup and running > performance: > 1. Merge calls, merge writeU1 and writeU2 into writeU3 > 2. Merge calls, merge writeU1 and writeIndex operations > 3. Directly use writeU1 instead of writeBytecode > 4. Rewrite the implementation of load

Re: RFR: 8341141: Optimize DirectCodeBuilder [v14]

2024-09-30 Thread Chen Liang
On Mon, 30 Sep 2024 21:14:50 GMT, Shaojin Wen wrote: >> Some DirectCodeBuilder related optimizations to improve startup and running >> performance: >> 1. Merge calls, merge writeU1 and writeU2 into writeU3 >> 2. Merge calls, merge writeU1 and writeIndex operations >> 3. Directly use writeU1 inst

Re: RFR: 8341141: Optimize DirectCodeBuilder [v14]

2024-09-30 Thread Shaojin Wen
> Some DirectCodeBuilder related optimizations to improve startup and running > performance: > 1. Merge calls, merge writeU1 and writeU2 into writeU3 > 2. Merge calls, merge writeU1 and writeIndex operations > 3. Directly use writeU1 instead of writeBytecode > 4. Rewrite the implementation of load

Re: RFR: 8341141: Optimize DirectCodeBuilder [v13]

2024-09-30 Thread Shaojin Wen
> Some DirectCodeBuilder related optimizations to improve startup and running > performance: > 1. Merge calls, merge writeU1 and writeU2 into writeU3 > 2. Merge calls, merge writeU1 and writeIndex operations > 3. Directly use writeU1 instead of writeBytecode > 4. Rewrite the implementation of load

Re: RFR: 8341141: Optimize DirectCodeBuilder [v12]

2024-09-30 Thread Shaojin Wen
> Some DirectCodeBuilder related optimizations to improve startup and running > performance: > 1. Merge calls, merge writeU1 and writeU2 into writeU3 > 2. Merge calls, merge writeU1 and writeIndex operations > 3. Directly use writeU1 instead of writeBytecode > 4. Rewrite the implementation of load

Re: RFR: 8341141: Optimize DirectCodeBuilder [v10]

2024-09-30 Thread Shaojin Wen
On Mon, 30 Sep 2024 06:48:07 GMT, Shaojin Wen wrote: >> Some DirectCodeBuilder related optimizations to improve startup and running >> performance: >> 1. Merge calls, merge writeU1 and writeU2 into writeU3 >> 2. Merge calls, merge writeU1 and writeIndex operations >> 3. Directly use writeU1 inst

Re: RFR: 8341141: Optimize DirectCodeBuilder [v11]

2024-09-30 Thread Shaojin Wen
> Some DirectCodeBuilder related optimizations to improve startup and running > performance: > 1. Merge calls, merge writeU1 and writeU2 into writeU3 > 2. Merge calls, merge writeU1 and writeIndex operations > 3. Directly use writeU1 instead of writeBytecode > 4. Rewrite the implementation of load

Re: RFR: 8341141: Optimize DirectCodeBuilder [v10]

2024-09-29 Thread Shaojin Wen
> Some DirectCodeBuilder related optimizations to improve startup and running > performance: > 1. Merge calls, merge writeU1 and writeU2 into writeU3 > 2. Merge calls, merge writeU1 and writeIndex operations > 3. Directly use writeU1 instead of writeBytecode > 4. Rewrite the implementation of load

Re: RFR: 8341141: Optimize DirectCodeBuilder [v9]

2024-09-29 Thread Shaojin Wen
> Some DirectCodeBuilder related optimizations to improve startup and running > performance: > 1. Merge calls, merge writeU1 and writeU2 into writeU3 > 2. Merge calls, merge writeU1 and writeIndex operations > 3. Directly use writeU1 instead of writeBytecode > 4. Rewrite the implementation of load

Re: RFR: 8341141: Optimize DirectCodeBuilder [v8]

2024-09-29 Thread Shaojin Wen
> Some DirectCodeBuilder related optimizations to improve startup and running > performance: > 1. Merge calls, merge writeU1 and writeU2 into writeU3 > 2. Merge calls, merge writeU1 and writeIndex operations > 3. Directly use writeU1 instead of writeBytecode > 4. Rewrite the implementation of load

Re: RFR: 8341141: Optimize DirectCodeBuilder [v7]

2024-09-29 Thread Shaojin Wen
> Some DirectCodeBuilder related optimizations to improve startup and running > performance: > 1. Merge calls, merge writeU1 and writeU2 into writeU3 > 2. Merge calls, merge writeU1 and writeIndex operations > 3. Directly use writeU1 instead of writeBytecode > 4. Rewrite the implementation of load

Re: RFR: 8341141: Optimize DirectCodeBuilder [v6]

2024-09-29 Thread Shaojin Wen
> Some DirectCodeBuilder related optimizations to improve startup and running > performance: > 1. Merge calls, merge writeU1 and writeU2 into writeU3 > 2. Merge calls, merge writeU1 and writeIndex operations > 3. Directly use writeU1 instead of writeBytecode > 4. Rewrite the implementation of load

Re: RFR: 8341141: Optimize DirectCodeBuilder [v5]

2024-09-29 Thread Shaojin Wen
> Some DirectCodeBuilder related optimizations to improve startup and running > performance: > 1. Merge calls, merge writeU1 and writeU2 into writeU3 > 2. Merge calls, merge writeU1 and writeIndex operations > 3. Directly use writeU1 instead of writeBytecode > 4. Rewrite the implementation of load

Re: RFR: 8341141: Optimize DirectCodeBuilder [v4]

2024-09-29 Thread Chen Liang
On Sun, 29 Sep 2024 13:35:14 GMT, Shaojin Wen wrote: >> Some DirectCodeBuilder related optimizations to improve startup and running >> performance: >> 1. Merge calls, merge writeU1 and writeU2 into writeU3 >> 2. Merge calls, merge writeU1 and writeIndex operations >> 3. Directly use writeU1 inst

Re: RFR: 8341141: Optimize DirectCodeBuilder [v4]

2024-09-29 Thread Claes Redestad
On Sun, 29 Sep 2024 13:35:14 GMT, Shaojin Wen wrote: >> Some DirectCodeBuilder related optimizations to improve startup and running >> performance: >> 1. Merge calls, merge writeU1 and writeU2 into writeU3 >> 2. Merge calls, merge writeU1 and writeIndex operations >> 3. Directly use writeU1 inst

Re: RFR: 8341141: Optimize DirectCodeBuilder [v4]

2024-09-29 Thread Shaojin Wen
> Some DirectCodeBuilder related optimizations to improve startup and running > performance: > 1. Merge calls, merge writeU1 and writeU2 into writeU3 > 2. Merge calls, merge writeU1 and writeIndex operations > 3. Directly use writeU1 instead of writeBytecode > 4. Rewrite the implementation of load

Re: RFR: 8341141: Optimize DirectCodeBuilder [v3]

2024-09-29 Thread Shaojin Wen
> Some DirectCodeBuilder related optimizations to improve startup and running > performance: > 1. Merge calls, merge writeU1 and writeU2 into writeU3 > 2. Merge calls, merge writeU1 and writeIndex operations > 3. Directly use writeU1 instead of writeBytecode > 4. Rewrite the implementation of load

Re: RFR: 8341141: Optimize DirectCodeBuilder [v2]

2024-09-29 Thread Shaojin Wen
> Some DirectCodeBuilder related optimizations to improve startup and running > performance: > 1. Merge calls, merge writeU1 and writeU2 into writeU3 > 2. Merge calls, merge writeU1 and writeIndex operations > 3. Directly use writeU1 instead of writeBytecode > 4. Rewrite the implementation of load

RFR: 8341141: Optimize DirectCodeBuilder

2024-09-28 Thread Shaojin Wen
Some DirectCodeBuilder related optimizations to improve startup and running performance: 1. Merge calls, merge writeU1 and writeU2 into writeU3 2. Merge calls, merge writeU1 and writeIndex operations 3. Directly use writeU1 instead of writeBytecode 4. Rewrite the implementation of load and store