Re: RFR: 8318650: Optimized subword gather for x86 targets. [v13]

2024-02-24 Thread Jatin Bhateja
On Tue, 20 Feb 2024 08:36:29 GMT, Emanuel Peter wrote: >> Jatin Bhateja has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Review comments resolutions. > > src/hotspot/cpu/x86/x86.ad line 4120: > >> 4118: BasicType elem_bt =

Re: RFR: 8318650: Optimized subword gather for x86 targets. [v14]

2024-02-24 Thread Jatin Bhateja
> Hi All, > > This patch optimizes sub-word gather operation for x86 targets with AVX2 and > AVX512 features. > > Following is the summary of changes:- > > 1) Intrinsify sub-word gather using hybrid algorithm which initially > partially unrolls scalar loop to accumulates values from gather

Re: RFR: 8318650: Optimized subword gather for x86 targets. [v13]

2024-02-24 Thread Jatin Bhateja
On Tue, 20 Feb 2024 08:04:27 GMT, Emanuel Peter wrote: >> src/hotspot/cpu/x86/c2_MacroAssembler_x86.cpp line 1584: >> >>> 1582: Label *larr[] = {, , , }; >>> 1583: for (int i = 0; i < 4; i++) { >>> 1584: // dst[i] = mask ? src[index[i]] : 0 >> >> I like these comments a lot! >>

Re: RFR: JDK-8316708: Augment WorstCaseTests with more cases [v3]

2024-02-24 Thread Joe Darcy
On Fri, 23 Feb 2024 19:18:22 GMT, Joe Darcy wrote: > > Both `Math.cos` and `StrictMath.cos` produce the correctly rounded result > > here. I don't know why the paper says otherwise. Perhaps OpenLibm is not > > exactly fdlibm. > > I've looked a bit over the OpenLibm changelog. They've added a

Re: RFR: JDK-8316708: Augment WorstCaseTests with more cases [v3]

2024-02-24 Thread Joe Darcy
> A new paper > > "Accuracy of Mathematical Functions in Single, Double, Double Extended, and > Quadruple Precision" > by Brian Gladman, Vincenzo Innocente and Paul Zimmermann > https://members.loria.fr/PZimmermann/papers/accuracy.pdf > > details the inputs with generate the worst-case

Re: RFR: 8321156: Improve the handling of invalid UTF-8 byte sequences for ZipInputStream::getNextEntry and ZipFile::getComment

2024-02-24 Thread Lance Andersen
On Sat, 24 Feb 2024 17:15:02 GMT, Eirik Bjørsnøs wrote: > Since the CSR is already approved, I'll add a question here: > > `ZipFile` performs a lot of validation while opening ZIP files, including > throwning ZipException for invalid entry names or comments. Why handle the > ZIP file comment

Re: RFR: 8321156: Improve the handling of invalid UTF-8 byte sequences for ZipInputStream::getNextEntry and ZipFile::getComment [v2]

2024-02-24 Thread Lance Andersen
On Sat, 24 Feb 2024 16:57:50 GMT, Eirik Bjørsnøs wrote: >> Lance Andersen has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Updates based on 1st round of feedback > > src/java.base/share/classes/java/util/zip/ZipFile.java line 331: > >>

Re: RFR: 8321156: Improve the handling of invalid UTF-8 byte sequences for ZipInputStream::getNextEntry and ZipFile::getComment [v2]

2024-02-24 Thread Lance Andersen
On Sat, 24 Feb 2024 18:51:22 GMT, Lance Andersen wrote: >> Please review this PR which addresses the handling of invalid UTF-8 byte >> sequences in the entry name of a LOC file header and a Zip file comment >> which is returned via ZipFile::getComment. >> >> As part of the change,

Re: RFR: 8321156: Improve the handling of invalid UTF-8 byte sequences for ZipInputStream::getNextEntry and ZipFile::getComment [v2]

2024-02-24 Thread Lance Andersen
> Please review this PR which addresses the handling of invalid UTF-8 byte > sequences in the entry name of a LOC file header and a Zip file comment which > is returned via ZipFile::getComment. > > As part of the change, `ZipFile::getComment` will now return `null` if an > invalid UTF-8 byte

Re: RFR: 8174269: Remove COMPAT locale data provider from JDK

2024-02-24 Thread Andrey Turbanov
On Fri, 23 Feb 2024 21:24:10 GMT, Naoto Sato wrote: > This PR intends to remove the legacy `COMPAT` locale data from the JDK. The > `COMPAT` locale data was introduced for applications' migratory purposes > transitioning to `CLDR`. It is becoming a technical debt and now is the time > to

RFR: 8326617: String Template FMT removes unnecessary int to long type cast

2024-02-24 Thread Shaojin Wen
In the current version, FMT."v =%d{1}" will call the StringConcatHelper.prepend(long/byte[]/long) method, which should behave the same as STR."v ={1}". Call StringConcatHelper.prepend(long/byte[]/int), should not convert int to long Please review and don't hesitate to critique my approach and

Re: RFR: 8321156: Improve the handling of invalid UTF-8 byte sequences for ZipInputStream::getNextEntry and ZipFile::getComment

2024-02-24 Thread Eirik Bjørsnøs
On Sat, 24 Feb 2024 14:56:01 GMT, Lance Andersen wrote: > Please review this PR which addresses the handling of invalid UTF-8 byte > sequences in the entry name of a LOC file header and a Zip file comment which > is returned via ZipFile::getComment. > > As part of the change,

Re: RFR: 8321156: Improve the handling of invalid UTF-8 byte sequences for ZipInputStream::getNextEntry and ZipFile::getComment

2024-02-24 Thread Eirik Bjørsnøs
On Sat, 24 Feb 2024 14:56:01 GMT, Lance Andersen wrote: > Please review this PR which addresses the handling of invalid UTF-8 byte > sequences in the entry name of a LOC file header and a Zip file comment which > is returned via ZipFile::getComment. > > As part of the change,

Re: RFR: 8326096: Deprecate getTotalIn, getTotalOut methods of java.util.zip.Inflater, java.util.zip.Deflater [v7]

2024-02-24 Thread Eirik Bjørsnøs
> Please review this PR which proposes that we officially deprecate the > following four methods in the `java.util.zip` package: > > * `Inflater.getTotalIn()` > * `Inflater.getTotalOut()` > * `Deflater.getTotalIn()` > * `Deflater.getTotalOut()` > > Since these legacy methods return `int`, they

Re: RFR: JDK-8326389: [test] improve assertEquals failure output [v3]

2024-02-24 Thread Christoph Langer
On Sat, 24 Feb 2024 11:45:46 GMT, Jaikiran Pai wrote: > This is similar to what other test libraries usually report for such failures. But in the case of a non-empty `msg` you would not see the actual values any more which I think could be helpful in a lot of cases... - PR

RFR: 8321156: Improve the handling of invalid UTF-8 byte sequences for ZipInputStream::getNextEntry and ZipFile::getComment

2024-02-24 Thread Lance Andersen
Please review this PR which addresses the handling of invalid UTF-8 byte sequences in the entry name of a LOC file header and a Zip file comment which is returned via ZipFile::getComment. As part of the change, `ZipFile::getComment` will now return `null` if an invalid UTF-8 byte sequence is

Re: RFR: JDK-8326389: [test] improve assertEquals failure output [v3]

2024-02-24 Thread Jaikiran Pai
On Thu, 22 Feb 2024 16:01:19 GMT, Matthias Baesken wrote: >> Currently assertEquals has in the failure case sometimes confusing output >> like : >> >> java.lang.RuntimeException: VM output should contain exactly one RTM locking >> statistics entry for method >>

Re: RFR: 8174269: Remove COMPAT locale data provider from JDK

2024-02-24 Thread Alan Bateman
On Fri, 23 Feb 2024 21:24:10 GMT, Naoto Sato wrote: > This PR intends to remove the legacy `COMPAT` locale data from the JDK. The > `COMPAT` locale data was introduced for applications' migratory purposes > transitioning to `CLDR`. It is becoming a technical debt and now is the time > to