Re: RFR: 8299748: java/util/zip/Deinflate.java failing on s390x [v2]

2023-03-05 Thread Jaikiran Pai
On Thu, 2 Feb 2023 08:27:55 GMT, Amit Kumar wrote: >> DeInflate.java test fails on s390x platform because size for out1 array >> which is responsible for storing the compressed data is insufficient. And >> being unable to write whole compressed data on array, on s390 whole data >> can't be

Re: RFR: 8299748: java/util/zip/Deinflate.java failing on s390x [v2]

2023-03-05 Thread Amit Kumar
On Thu, 2 Feb 2023 08:27:55 GMT, Amit Kumar wrote: >> DeInflate.java test fails on s390x platform because size for out1 array >> which is responsible for storing the compressed data is insufficient. And >> being unable to write whole compressed data on array, on s390 whole data >> can't be

Re: RFR: JDK-8302801: Remove fdlibm C sources [v4]

2023-03-05 Thread Vladimir Kozlov
On Sun, 5 Mar 2023 06:19:06 GMT, Joe Darcy wrote: >> While the review of https://github.com/openjdk/jdk/pull/12800 finishes up, I >> thought I'd get out for the review the next phase of the FDLIBM port: >> removing the FDLIBM C sources from the repo. >> >> A repo with the changes for

Re: UUID creation performance

2023-03-05 Thread Brett Okken
The new ByteArray class works great for the nameUUIDFromBytes method, which must be in big endian. For randomUUID, byte order does not matter, so using native would be fastest, but there does not appear to be a utility class for that. Is there a preference of just having a native order VarHandle

Re: RFR: JDK-8285932 Implementation of JEP 430 String Templates (Preview) [v42]

2023-03-05 Thread Marius Volkhart
On Mon, 27 Feb 2023 12:47:03 GMT, Jim Laskey wrote: >> Enhance the Java programming language with string templates, which are >> similar to string literals but contain embedded expressions. A string >> template is interpreted at run time by replacing each expression with the >> result of

Re: RFR: 8303431: [JVMCI] libgraal annotation API [v3]

2023-03-05 Thread Doug Simon
> This PR extends JVMCI with new API (`jdk.vm.ci.meta.Annotated`) for accessing > annotations. The main differences from `java.lang.reflect.AnnotatedElement` > are: > * Each `Annotated` method explicitly specifies the annotation type(s) for > which it wants annotation data. That is, there is no

Re: RFR: JDK-8302801: Remove fdlibm C sources [v4]

2023-03-05 Thread David Holmes
On Sun, 5 Mar 2023 06:19:06 GMT, Joe Darcy wrote: >> While the review of https://github.com/openjdk/jdk/pull/12800 finishes up, I >> thought I'd get out for the review the next phase of the FDLIBM port: >> removing the FDLIBM C sources from the repo. >> >> A repo with the changes for

Re: RFR: 8303431: [JVMCI] libgraal annotation API [v2]

2023-03-05 Thread Doug Simon
> This PR extends JVMCI with new API (`jdk.vm.ci.meta.Annotated`) for accessing > annotations. The main differences from `java.lang.reflect.AnnotatedElement` > are: > * Each `Annotated` method explicitly specifies the annotation type(s) for > which it wants annotation data. That is, there is no

Re: RFR: JDK-8302801: Remove fdlibm C sources [v4]

2023-03-05 Thread Alan Bateman
On Sun, 5 Mar 2023 06:19:06 GMT, Joe Darcy wrote: >> While the review of https://github.com/openjdk/jdk/pull/12800 finishes up, I >> thought I'd get out for the review the next phase of the FDLIBM port: >> removing the FDLIBM C sources from the repo. >> >> A repo with the changes for

Re: RFR: JDK-8302801: Remove fdlibm C sources [v4]

2023-03-05 Thread Joe Darcy
On Thu, 2 Mar 2023 18:27:09 GMT, Joe Darcy wrote: >> make/autoconf/buildjdk-spec.gmk.in line 85: >> >>> 83: JVM_LIBS := @OPENJDK_BUILD_JVM_LIBS@ >>> 84: >>> 85: FDLIBM_CFLAGS := @OPENJDK_BUILD_FDLIBM_CFLAGS@ >> >> If the hotspot build still needs `FDLIBM_CFLAGS`, then this line needs to >>

Re: RFR: JDK-8302027: Port fdlibm trig functions (sin, cos, tan) to Java [v6]

2023-03-05 Thread Andrey Turbanov
The message from this sender included one or more files which could not be scanned for virus detection; do not open these files unless you are certain of the sender's intent. -- On Wed, 1 Mar 2023 14:26:44 GMT, Andrey Turbanov