RFR: 8156916: intrinsify MethodHandles.arrayConstructor()

2023-04-29 Thread Chen Liang
Try to intrinsify MethodHandles.arrayConstructor for java.lang, java.lang.invoke, and java.util types, which InvokerBytecodeGenerator can refer to directly in a class constant. In fact, the performance difference of Array.newInstance and anewarray is quite negligible per benchmark: Benchmark

Re: RFR: JDK-8306112 Implementation of JEP 445: Flexible Main Methods and Anonymous Main Classes (Preview) [v7]

2023-04-29 Thread Jim Laskey
> Add flexible main methods and anonymous main classes to the Java language. Jim Laskey has updated the pull request incrementally with two additional commits since the last revision: - Move AnonymousMainClass to parser - Revert java launch - Changes: - all: https://git.openjdk

Re: RFR: JDK-8266431: Dual-Pivot Quicksort improvements (Radix sort) [v6]

2023-04-29 Thread iaroslavski
On Sun, 23 Apr 2023 17:33:38 GMT, Laurent Bourgès wrote: >> * Optimized mixed insertion sort >> * Optimized insertion sort >> * Optimized Radix sort >> * Updated microbenchmark >> >> I am going on previous PR by Vladimir Yaroslavskyi: >> https://github.com/openjdk/jdk/pull/3938 > >

Re: RFR: JDK-8077371: Binary files in JAXP test should be removed [v4]

2023-04-29 Thread Lance Andersen
On Thu, 27 Apr 2023 17:35:12 GMT, Mahendra Chhipa wrote: >> Test is updated to create the binary files during test execution. > > Mahendra Chhipa has updated the pull request incrementally with one > additional commit since the last revision: > > Implemented the review comments. test/jdk/jav

Integrated: 8307088: Allow the jdbc.drivers system property to be searchable

2023-04-29 Thread Lance Andersen
On Fri, 28 Apr 2023 17:28:47 GMT, Lance Andersen wrote: > Hi all, > > Please review this trivial change which allows the` jdbc.drivers` system > property to be searchable. > > Best. > Lance This pull request has now been integrated. Changeset: bb7608ba Author:Lance Andersen URL:

The introduction of Sequenced collections is not a source compatible change

2023-04-29 Thread Remi Forax
I've several repositories that now fails to compile with the latest jdk21, which introduces sequence collections. The introduction of a common supertype to existing collections is *not* a source compatible change because of type inference. Here is a simplified example: public static void m(L

Integrated: 8304837: Classfile API throws IOOBE for MethodParameters attribute without parameter names

2023-04-29 Thread Hannes Greule
On Thu, 23 Mar 2023 19:45:21 GMT, Hannes Greule wrote: > After merging master into https://github.com/openjdk/jdk/pull/9862, we > encountered test failures (e.g., > https://github.com/SirYwell/jdk/actions/runs/4500940829/jobs/7923018438#step:9:2541). > The Classfile API tries to read from cons

Re: RFR: 8304837: Classfile API throws IOOBE for MethodParameters attribute without parameter names [v3]

2023-04-29 Thread Julian Waters
On Fri, 24 Mar 2023 06:42:30 GMT, Hannes Greule wrote: >> After merging master into https://github.com/openjdk/jdk/pull/9862, we >> encountered test failures (e.g., >> https://github.com/SirYwell/jdk/actions/runs/4500940829/jobs/7923018438#step:9:2541). >> The Classfile API tries to read from