On Fri, 26 Apr 2024 14:06:02 GMT, Hamlin Li wrote:
> Hi,
> Can you help to review this simple patch?
> Some index check in Byte/ShortVector.fromArray/fromArray0Template seems not
> necessary, could be removed.
> Thanks
In one side, yes, there is a gap in the tests.
In another
Hi,
Can you help to review this patch?
Thanks!
This is similar with previous JDK-8334396.
Added some tests.
### Test
| Tests | Scores | Errors | Unit
-- | -- | -- | -- | --
Intrinsic, +zbb, +rvv | Characters.reverseBytes | 1654.535 | 69.36 | ns/op
| Shorts.reverseBytes | 1795.403 | 44.015 |
On Mon, 24 Jun 2024 21:01:14 GMT, Ludovic Henry wrote:
>> Hi,
>> Can you help to review this patch?
>> Thanks!
>>
>> This is similar with previous JDK-8334396.
>> Added some tests.
>>
>> ### Test
>>
>> | Tests | Scores | Errors | Unit
>> -- | -- | -- | -- | --
>> Intrinsic, +zbb, +rvv | Char
On Tue, 25 Jun 2024 12:51:46 GMT, Ludovic Henry wrote:
>> Hi,
>> Can you help to review this patch?
>> Thanks!
>>
>> This is similar with previous JDK-8334396.
>> Added some tests.
>>
>> ### Test
>>
>> | Tests | Scores | Errors | Unit
>> -- | -- | -- | -- | --
>> Intrinsic, +zbb, +rvv | Char
On Fri, 21 Jun 2024 14:24:26 GMT, Hamlin Li wrote:
> Hi,
> Can you help to review this patch?
> Thanks!
>
> This is similar with previous JDK-8334396.
> Added some tests.
>
> ### Test
>
> | Tests | Scores | Errors | Unit
> -- | -- | --
Hi,
Can you help to review this simple patch?
Currently, lineSize linesize for MIME case in macro bench test Base64Decode is
only "4", but in Base64.Encoder default linesize for MIME encoder is 76.
It's helpful to add more linesize, e.g. 76 and so on.
Thanks!
-
Commit messages:
-
On Mon, 19 Aug 2024 16:43:02 GMT, Hamlin Li wrote:
> Hi,
> Can you help to review this simple patch?
>
> Currently, lineSize linesize for MIME case in macro bench test Base64Decode
> is only "4", but in Base64.Encoder default linesize for MIME encoder is 76.
> It&#x
On Wed, 21 Aug 2024 10:03:57 GMT, Robbin Ehn wrote:
>> Hi,
>> Can you help to review this simple patch?
>>
>> Currently, lineSize linesize for MIME case in macro bench test Base64Decode
>> is only "4", but in Base64.Encoder default linesize for MIME encoder is 76.
>> It's helpful to add more li
On Thu, 29 Jun 2023 21:17:49 GMT, Pavel Rappo wrote:
> Please review this IDE-assisted typo hunt, which I plan to backport to jdk21.
Marked as reviewed by mli (Reviewer).
-
PR Review: https://git.openjdk.org/jdk/pull/14718#pullrequestreview-1506596503
On Sat, 1 Jul 2023 07:53:17 GMT, Swati Sharma wrote:
> The below benchmark files have scaling issues due to cache contention and
> leads to poor scaling when run on multiple threads. The patch sets the scope
> from benchmark level to thread level to fix the issue:
> - org/openjdk/bench/java/io/
s mean that @State only matters when the annotated class is used as
>> a parameter of a @Benchmark method, but in the tests you modifed, seems
>> there is no such use case.
>> Please also check the sample usages at
>> https://github.com/openjdk/jmh/blob/master/jmh-samples/
On Sat, 1 Jul 2023 07:53:17 GMT, Swati Sharma wrote:
> The below benchmark files have scaling issues due to cache contention and
> leads to poor scaling when run on multiple threads. The patch sets the scope
> from benchmark level to thread level to fix the issue:
> - org/openjdk/bench/java/io/
On Mon, 10 Jul 2023 08:17:59 GMT, Hamlin Li wrote:
>> @Hamlin-Li
>> The PR is fully correct.
>> Don't forget, every Java instance method has a specific argument which
>> called "this". That is why @State annotation is working.
>
> @kuksenko @swati
On Fri, 6 Oct 2023 21:07:51 GMT, Justin Lu wrote:
>> Please review this PR which cleans up the static test utility class
>> _HexDumpReader_.
>>
>> This cleans up the code by replacing the nested _ByteArrayBuilder_ class
>> with _HexFormat_, and simplifies the File processing by using a stream.
On Thu, 5 Oct 2023 17:57:00 GMT, Naoto Sato wrote:
> Adding IOOBE clause to clarify the behavior on an invalid `parsePos` argument
> on calling `ListFormat::parseObject()`. A corresponding CSR has also been
> drafted.
Marked as reviewed by mli (Reviewer).
-
PR Review: https://git
On Mon, 16 Oct 2023 01:18:52 GMT, David Holmes wrote:
> The open nroff manpage files are out of sync with their closed markdown
> sources and need to be refreshed.
>
> The changes to apply are coming from:
>
> - [JDK-8288158](https://bugs.openjdk.org/browse/JDK-8288158): Add an anchor
> to ea
On Wed, 18 Oct 2023 17:35:58 GMT, Ilya Gavrilin wrote:
>> Hi all, please review this changes into risc-v floating point copysign and
>> signum intrinsics.
>> CopySign - returns first argument with the sign of second. On risc-v we have
>> `fsgnj.x` instruction, which can implement this intrinsic
On Wed, 11 Oct 2023 05:29:22 GMT, ANUPAM DEV wrote:
>> Hello,
>>
>> I have fixed the typo in the comment for the constructor
>> IdentityHashMap(Map m)
>>
>> before: Constructs a new identity hash map containing the keys-value
>> mappings in the specified map.
>> after: Constructs a new identi
On Wed, 11 Oct 2023 11:18:41 GMT, Per Minborg wrote:
>> This PR proposes to add @sealedGraph to MethodHandleDesc and descendants.
>
> Per Minborg has updated the pull request incrementally with two additional
> commits since the last revision:
>
> - Update copyright year
> - Revoke change in
On Sat, 21 Oct 2023 10:56:01 GMT, Doug Simon wrote:
>> The Graal code base has
>> [renamed](https://github.com/oracle/graal/commit/1e41203d10db321f86723eac90f6cd0573b08b33)
>> its module to `jdk.compiler.graal` as part of preparations for Project
>> Galahad. Due to the way Java modules work, t
On Fri, 20 Oct 2023 17:57:46 GMT, Naoto Sato wrote:
> Proposing adding getter methods for `locale` and `patterns` fields in the
> `ListFormat` instances. Those fields are used for `equals()`, but without the
> public getter methods, users cannot tell the reasoning of the equality of two
> inst
On Tue, 31 Oct 2023 07:43:43 GMT, Leo Korinth wrote:
> Changing from `Class c` to `Class c` removes two warnings.
LGTM
-
Marked as reviewed by mli (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/16431#pullrequestreview-1706582873
On Sat, 3 Feb 2024 14:36:14 GMT, Claes Redestad wrote:
>> This patch streamlines and specializes various `String::indexOf` methods.
>> Mainly avoids the need for clamping and doing checks that are redundant in
>> almost all cases, moving the checks to the API boundary where they are
>> needed.
On Thu, 7 Dec 2023 09:30:01 GMT, Xiaohong Gong wrote:
>> Currently the vector floating-point math APIs like
>> `VectorOperators.SIN/COS/TAN...` are not intrinsified on AArch64 platform,
>> which causes large performance gap on AArch64. Note that those APIs are
>> optimized by C2 compiler on X8
On Thu, 7 Dec 2023 09:30:01 GMT, Xiaohong Gong wrote:
>> Currently the vector floating-point math APIs like
>> `VectorOperators.SIN/COS/TAN...` are not intrinsified on AArch64 platform,
>> which causes large performance gap on AArch64. Note that those APIs are
>> optimized by C2 compiler on X8
On Fri, 1 Mar 2024 15:10:30 GMT, Magnus Ihse Bursie wrote:
>> Xiaohong Gong has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Fix potential attribute issue
>
> Iirc, your assessment is right; the code should be ready for integration; I
>
Hi,
Can you help to review this simple patch?
Some index check in Byte/ShortVector.fromArray/fromArray0Template seems not
necessary, could be removed.
Thanks
-
Commit messages:
- Initial commit
Changes: https://git.openjdk.org/jdk/pull/18977/files
Webrev: https://webrevs.openjdk.
On Tue, 23 Apr 2024 09:46:08 GMT, Hamlin Li wrote:
>> HI,
>> Can you have a look at this patch adding some tests for Math.round
>> instrinsics?
>> Thanks!
>>
>> ### FYI:
>> During the development of RoundVF/RoundF, we faced the issues which were
&g
> please correct me if I'm assume incorrectly.
Hamlin Li has updated the pull request incrementally with one additional commit
since the last revision:
fix issues; modify vm options to make sure test the expected behaviors.
-
Changes:
- all: https://git.openjdk.org/jdk/pull
On Sun, 28 Apr 2024 11:34:57 GMT, Andrew Haley wrote:
>> Hamlin Li has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Add vectorized and scalar version Float tests checking full 32 bits range
>
> test/hotspo
On Tue, 7 May 2024 13:36:55 GMT, Emanuel Peter wrote:
>> test/hotspot/jtreg/compiler/floatingpoint/TestRoundFloatAll.java line 31:
>>
>>> 29: * @library /test/lib /
>>> 30: * @modules java.base/jdk.internal.math
>>> 31: * @run main/othervm -XX:-TieredCompilation
>>> -XX:CompileThresholdScali
> please correct me if I'm assume incorrectly.
Hamlin Li has updated the pull request incrementally with one additional commit
since the last revision:
misc fixes
-
Changes:
- all: https://git.openjdk.org/jdk/pull/17753/files
- new: https://git.openjdk.org/jdk/pull/1
On Tue, 7 May 2024 13:30:12 GMT, Emanuel Peter wrote:
>> Hamlin Li has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> fix issues; modify vm options to make sure test the expected behaviors.
>
> test/hotspo
On Tue, 7 May 2024 13:44:06 GMT, Emanuel Peter wrote:
> Thanks for the extra tests!
>
Thanks for reviewing.
> Can you measure how much time each test now takes on your machine?
>
Only TestRoundVectorFloatAll.java took longer, but still in one minute, others
run rather quicker than it.
> I
On Mon, 20 May 2024 19:21:44 GMT, Paul Sandoz wrote:
>> Hi,
>> Can you help to review this simple patch?
>> Some index check in Byte/ShortVector.fromArray/fromArray0Template seems not
>> necessary, could be removed.
>> Thanks
>
> That does not look correct and will only check a prefix indexes. A
On Thu, 17 Oct 2024 22:44:05 GMT, Mandy Chung wrote:
>> The old core reflection implementation generates dynamic classes that are
>> special cases in the VM to bypass bytecode verification to workaround
>> various issues [1] [2] [3].
>>
>> The old core reflection implementation was [removed in
On Mon, 9 Dec 2024 21:02:03 GMT, Magnus Ihse Bursie wrote:
>> Some files have been modified in 2024, but the copyright year has not been
>> properly updated. This should be fixed.
>>
>> I have located these modified files using:
>>
>> git log --since="Jan 1" --name-only --pretty=format: | sor
On Mon, 9 Dec 2024 15:42:57 GMT, Magnus Ihse Bursie wrote:
>> Some files have been modified in 2024, but the copyright year has not been
>> properly updated. This should be fixed.
>>
>> I have located these modified files using:
>>
>> git log --since="Jan 1" --name-only --pretty=format: | sor
On Fri, 17 Jan 2025 16:02:55 GMT, Jatin Bhateja wrote:
>> Hi All,
>>
>> This patch adds C2 compiler support for various Float16 operations added by
>> [PR#22128](https://github.com/openjdk/jdk/pull/22128)
>>
>> Following is the summary of changes included with this patch:-
>>
>> 1. Detection
>const_oop() == nullptr) {
>>
>> Hi, this is not a review comment.
>> Just curious, to continue the following code path why does `box_type` must
>> have a valid `const_oop`?
>
> @Hamlin-Li , Class types are passed as constant oop, this check is added for
> argum
On Fri, 17 Jan 2025 16:02:55 GMT, Jatin Bhateja wrote:
>> Hi All,
>>
>> This patch adds C2 compiler support for various Float16 operations added by
>> [PR#22128](https://github.com/openjdk/jdk/pull/22128)
>>
>> Following is the summary of changes included with this patch:-
>>
>> 1. Detection
On Thu, 27 Mar 2025 14:32:18 GMT, Robbin Ehn wrote:
>> Hi, for you to consider.
>>
>> These tests constantly fails in qemu-user.
>> Either the require host to be same arch or they are very very slow in
>> emulation.
>> E.g. "ptrace(PTRACE_ATTACH, ..) failed for 405157: Function not
>> implemen
On Tue, 8 Apr 2025 07:07:12 GMT, Fei Yang wrote:
> Maybe we should check UseZicond and only enable UseCMoveUnconditionally &
> ConditionalMoveLimit conditionally?
Not sure what do you mean here.
> I don't see how enabling CMove will bring us any performance benefit without
> Zicond. It's done
On Mon, 7 Apr 2025 14:23:52 GMT, Hamlin Li wrote:
> Hi,
> Can you help to review this patch?
> On riscv, CMoveI/L already were implemented, but there are some gap:
> 1. CMoveI/L does not support comparison with float/double, corresponding
> tests are not turn on either.
> 2.
0.57 | 118.639 | 119.373 | 0.451 | ns/op |
> 1.002
> o.o.b.vm.compiler.IfMinMax.testVectorInt | avgt | 40 | 60130.165 | 60062.303
> | 61839.776 | 61895.194 | 15887.398 | 15924.502 | 15874.835 | 15667.936 |
> 101.94 | ns/op | 3.785
> o.o.b.vm.compiler.IfMinMax.testVectorLong | avgt |
On Wed, 9 Apr 2025 06:52:26 GMT, Fei Yang wrote:
>> This is to not enable Zicond automatically, but user can still turn it on
>> manually if they want to try or make sure it bring benefit on the specific
>> hardware.
>> Currently it's based on bananapi result, so maybe in the future we should
0.57 | 118.639 | 119.373 | 0.451 | ns/op |
> 1.002
> o.o.b.vm.compiler.IfMinMax.testVectorInt | avgt | 40 | 60130.165 | 60062.303
> | 61839.776 | 61895.194 | 15887.398 | 15924.502 | 15874.835 | 15667.936 |
> 101.94 | ns/op | 3.785
> o.o.b.vm.compiler.IfMinMax.testVectorLong | av
0.57 | 118.639 | 119.373 | 0.451 | ns/op |
> 1.002
> o.o.b.vm.compiler.IfMinMax.testVectorInt | avgt | 40 | 60130.165 | 60062.303
> | 61839.776 | 61895.194 | 15887.398 | 15924.502 | 15874.835 | 15667.936 |
> 101.94 | ns/op | 3.785
> o.o.b.vm.compiler.IfMinMax.testVectorLong | avgt |
0.57 | 118.639 | 119.373 | 0.451 | ns/op |
> 1.002
> o.o.b.vm.compiler.IfMinMax.testVectorInt | avgt | 40 | 60130.165 | 60062.303
> | 61839.776 | 61895.194 | 15887.398 | 15924.502 | 15874.835 | 15667.936 |
> 101.94 | ns/op | 3.785
> o.o.b.vm.compiler.IfMinMax.testVectorLong
On Wed, 9 Apr 2025 06:41:51 GMT, Fei Yang wrote:
> Sorry for not being clear enough. I am suggesting this:
if (UseZicond) {
FLAG_SET_DEFAULT(ConditionalMoveLimit, 3);
}
I think this depends on whether we should enable ConditionalMoveLimit based on
`UseZicond`? So, I'll leave it until
0.57 | 118.639 | 119.373 | 0.451 | ns/op |
> 1.002
> o.o.b.vm.compiler.IfMinMax.testVectorInt | avgt | 40 | 60130.165 | 60062.303
> | 61839.776 | 61895.194 | 15887.398 | 15924.502 | 15874.835 | 15667.936 |
> 101.94 | ns/op | 3.785
> o.o.b.vm.compiler.IfMinMax.testVectorLong |
On Tue, 8 Apr 2025 15:02:27 GMT, Feilong Jiang wrote:
>> Hamlin Li 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 five
0.57 | 118.639 | 119.373 | 0.451 | ns/op |
> 1.002
> o.o.b.vm.compiler.IfMinMax.testVectorInt | avgt | 40 | 60130.165 | 60062.303
> | 61839.776 | 61895.194 | 15887.398 | 15924.502 | 15874.835 | 15667.936 |
> 101.94 | ns/op | 3.785
> o.o.b.vm.compiler.IfMinMax.testV
On Fri, 11 Apr 2025 04:07:45 GMT, Fei Yang wrote:
>> Hamlin Li has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> enable more test
>
> src/hotspot/cpu/riscv/c2_MacroAssembler_riscv.cpp line 2173:
>
>&g
Hi,
Can you help to review this patch?
On riscv, CMoveI/L already were implemented, but there are some gap:
1. CMoveI/L does not support comparison with float/double, corresponding tests
are not turn on either.
2. Some optimization of C2 is not turned on, e.g. `Phi -> CMove -> min_max`.
3. lack of
0.57 | 118.639 | 119.373 | 0.451 | ns/op |
> 1.002
> o.o.b.vm.compiler.IfMinMax.testVectorInt | avgt | 40 | 60130.165 | 60062.303
> | 61839.776 | 61895.194 | 15887.398 | 15924.502 | 15874.835 | 15667.936 |
> 101.94 | ns/op | 3.785
> o.o.b.vm.compiler.IfMinMax.tes
On Wed, 16 Apr 2025 00:05:44 GMT, Fei Yang wrote:
>> Hamlin Li has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> minors
>
> test/hotspot/jtreg/compiler/vectorapi/TestVectorTest.java line 2:
>
>>
On Mon, 7 Apr 2025 14:23:52 GMT, Hamlin Li wrote:
> Hi,
> Can you help to review this patch?
> On riscv, CMoveI/L already were implemented, but there are some gap:
> 1. CMoveI/L does not support comparison with float/double, corresponding
> tests are not turn on either.
> 2.
On Tue, 22 Apr 2025 16:34:21 GMT, Hamlin Li wrote:
>> Vladimir Ivanov 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 24 addi
On Thu, 17 Apr 2025 18:03:47 GMT, Vladimir Ivanov wrote:
>> Migrate Vector API math library (SVML and SLEEF) linkage from native code
>> (in JVM) to Java FFM API.
>>
>> Since FFM API doesn't support vector calling conventions yet, migration
>> affects only symbol lookup for now. But it still e
On Wed, 23 Apr 2025 23:55:50 GMT, Vladimir Ivanov wrote:
>> Does the following check catch `UseRVV == false` case on RISC-V?
>>
>> public boolean isSupported(Operator op, VectorSpecies vspecies) {
>> ...
>> int maxLaneCount =
>> VectorSupport.getMaxLaneCount(vspecies.elementType())
On Wed, 23 Apr 2025 23:54:01 GMT, Vladimir Ivanov wrote:
>> Migrate Vector API math library (SVML and SLEEF) linkage from native code
>> (in JVM) to Java FFM API.
>>
>> Since FFM API doesn't support vector calling conventions yet, migration
>> affects only symbol lookup for now. But it still e
On Tue, 29 Apr 2025 22:22:19 GMT, Vladimir Ivanov wrote:
> Can you point me, please, to the relevant parts of the discussions? I wasn't
> part of them.
Sorry, I can not find out this specific answer for you, as there were too long
discussion there, the whole process took several months.
> Bot
Hi,
Can you help to review this patch?
Before [JDK-8353786](https://bugs.openjdk.org/browse/JDK-8353786), when a
released jdk not supportting sleef (for any reason, e.g. low gcc version,
intrinsic not supported, rvv not supported, and so on) runs on machine support
vector operation (e.g. on ris
On Tue, 29 Apr 2025 02:40:01 GMT, SendaoYan wrote:
>> Hi,
>> Can you help to review this patch?
>>
>> Before [JDK-8353786](https://bugs.openjdk.org/browse/JDK-8353786), when a
>> released jdk not supportting sleef (for any reason, e.g. low gcc version,
>> intrinsic not supported, rvv not suppo
0.57 | 118.639 | 119.373 | 0.451 | ns/op |
> 1.002
> o.o.b.vm.compiler.IfMinMax.testVectorInt | avgt | 40 | 60130.165 | 60062.303
> | 61839.776 | 61895.194 | 15887.398 | 15924.502 | 15874.835 | 15667.936 |
> 101.94 | ns/op | 3.785
> o.o.b.vm.compiler.IfMinMax.testV
On Wed, 30 Apr 2025 19:41:52 GMT, Vladimir Ivanov wrote:
> Overall, it still looks like a JDK build issue to me. Hiding problems
> occurred during the build is not good. If some toolchains can't successfully
> build the library, the library shouldn't be included in JDK.
No, in riscv case (poss
On Wed, 23 Apr 2025 01:02:19 GMT, Vladimir Ivanov wrote:
>> Migrate Vector API math library (SVML and SLEEF) linkage from native code
>> (in JVM) to Java FFM API.
>>
>> Since FFM API doesn't support vector calling conventions yet, migration
>> affects only symbol lookup for now. But it still e
On Wed, 23 Apr 2025 08:40:44 GMT, Hamlin Li wrote:
>> Vladimir Ivanov has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Avoid thread state transition in VectorSupport_GetCPUFeatures
>
> src/jdk.incubator.vect
On Wed, 23 Apr 2025 00:23:46 GMT, Vladimir Ivanov wrote:
>> A simple fix could be:
>>
>> diff --git a/src/hotspot/os_cpu/linux_riscv/vm_version_linux_riscv.cpp
>> b/src/hotspot/os_cpu/linux_riscv/vm_version_linux_riscv.cpp
>> index 484a2a645aa..a785dc65c9e 100644
>> --- a/src/hotspot/os_cpu/lin
On Mon, 14 Apr 2025 03:24:18 GMT, Feilong Jiang wrote:
>> Hamlin Li has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> minors
>
> Looks good. Thanks!
Thank you @feilongjiang @RealFYang !
-
PR Co
On Mon, 14 Apr 2025 06:26:10 GMT, Fei Yang wrote:
>> Hamlin Li has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> minors
>
> test/hotspot/jtreg/compiler/lib/ir_framework/TestFramework.java line 149:
>
&g
On Wed, 30 Apr 2025 12:00:45 GMT, Robbin Ehn wrote:
>> Hi, for you to consider.
>>
>> These tests constantly fails in qemu-user.
>> Either the require host to be same arch explicit or implicit (sysroot).
>> E.g. "ptrace(PTRACE_ATTACH, ..) failed for 405157: Function not
>> implemented'" for SA
On Thu, 1 May 2025 18:27:24 GMT, Vladimir Ivanov wrote:
> I want to understand the issue with missing entries in vector math native
> libraries first before making a decision how to proceed.
>
> > I don't think it's a build issue, the jdk vendor can choose to support it
> > or not, it's just t
On Mon, 28 Apr 2025 10:34:49 GMT, Hamlin Li wrote:
> Hi,
> Can you help to review this patch?
>
> Before [JDK-8353786](https://bugs.openjdk.org/browse/JDK-8353786), when a
> released jdk not supportting sleef (for any reason, e.g. low gcc version,
> intrinsic not supported,
On Mon, 28 Apr 2025 10:34:49 GMT, Hamlin Li wrote:
> Hi,
> Can you help to review this patch?
>
> Before [JDK-8353786](https://bugs.openjdk.org/browse/JDK-8353786), when a
> released jdk not supportting sleef (for any reason, e.g. low gcc version,
> intrinsic not supported,
On Mon, 28 Apr 2025 10:34:49 GMT, Hamlin Li wrote:
> Hi,
> Can you help to review this patch?
>
> Before [JDK-8353786](https://bugs.openjdk.org/browse/JDK-8353786), when a
> released jdk not supportting sleef (for any reason, e.g. low gcc version,
> intrinsic not supported,
77 matches
Mail list logo
