Re: RFR: 8295302: Do not use ArrayList when LambdaForm has a single ClassData [v3]

2022-10-18 Thread Andrey Turbanov
On Mon, 17 Oct 2022 21:48:42 GMT, Ioi Lam wrote: >> Please review this small optimization. As shown in the JBS issue, most of >> the generated LambdaForm classes have a single ClassData, so we can get a >> small footprint/speed improvement. > > Ioi Lam has updated the pull request with a new ta

Re: RFR: 8295302: Do not use ArrayList when LambdaForm has a single ClassData [v3]

2022-10-18 Thread Mandy Chung
On Tue, 18 Oct 2022 05:21:47 GMT, Ioi Lam wrote: >> src/java.base/share/classes/java/lang/invoke/InvokerBytecodeGenerator.java >> line 346: >> >>> 344: >>> 345: /** >>> 346: * Returns an object to pass this.classData to the method >>> of the >> >> What about: >> >> Suggestion: >>

Re: RFR: 8295302: Do not use ArrayList when LambdaForm has a single ClassData [v3]

2022-10-17 Thread Ioi Lam
On Tue, 18 Oct 2022 02:03:06 GMT, Mandy Chung wrote: >> Ioi Lam has updated the pull request with a new target base due to a merge >> or a rebase. The incremental webrev excludes the unrelated changes brought >> in by the merge/rebase. The pull request contains four additional commits >> since

Re: RFR: 8295302: Do not use ArrayList when LambdaForm has a single ClassData [v3]

2022-10-17 Thread Mandy Chung
On Mon, 17 Oct 2022 21:48:42 GMT, Ioi Lam wrote: >> Please review this small optimization. As shown in the JBS issue, most of >> the generated LambdaForm classes have a single ClassData, so we can get a >> small footprint/speed improvement. > > Ioi Lam has updated the pull request with a new ta

Re: RFR: 8295302: Do not use ArrayList when LambdaForm has a single ClassData [v3]

2022-10-17 Thread Ioi Lam
> Please review this small optimization. As shown in the JBS issue, most of the > generated LambdaForm classes have a single ClassData, so we can get a small > footprint/speed improvement. Ioi Lam has updated the pull request with a new target base due to a merge or a rebase. The incremental we