Re: RFR: 8287442: Reduce list to array conversions in java.lang.invoke.MethodHandles [v3]

2022-06-03 Thread Jorn Vernee 
On Fri, 3 Jun 2022 17:05:19 GMT, Claes Redestad  wrote:

>> In preparation of #8855 this PR refactors the conversions from `List` to 
>> array and array to `List`, reducing the number of conversions when calling 
>> `MethodHandles.dropArguments` in particular. This remove about ~5% of 
>> allocations on the `StringConcatFactoryBootstraps` microbenchmark.
>
> Claes Redestad has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Jorn review: Drop boolean, rename internal variant dropArgumentsTrusted

Marked as reviewed by jvernee (Reviewer).

-

PR: https://git.openjdk.java.net/jdk/pull/8923

Re: RFR: 8287442: Reduce list to array conversions in java.lang.invoke.MethodHandles [v3]

2022-06-03 Thread Claes Redestad 
> In preparation of #8855 this PR refactors the conversions from `List` to 
> array and array to `List`, reducing the number of conversions when calling 
> `MethodHandles.dropArguments` in particular. This remove about ~5% of 
> allocations on the `StringConcatFactoryBootstraps` microbenchmark.

Claes Redestad has updated the pull request incrementally with one additional 
commit since the last revision:

  Jorn review: Drop boolean, rename internal variant dropArgumentsTrusted

-

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/8923/files
  - new: https://git.openjdk.java.net/jdk/pull/8923/files/15ff2125..c590f0dd

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk=8923=02
 - incr: https://webrevs.openjdk.java.net/?repo=jdk=8923=01-02

  Stats: 13 lines in 3 files changed: 0 ins; 3 del; 10 mod
  Patch: https://git.openjdk.java.net/jdk/pull/8923.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/8923/head:pull/8923

PR: https://git.openjdk.java.net/jdk/pull/8923