On Tue, 8 Oct 2024 14:56:56 GMT, Hamlin Li wrote:
>> Hi,
>> Can you help to review this patch?
>> Thanks!
>>
>> This patch is based on https://github.com/openjdk/jdk/pull/20781 which added
>> the sleef source (in particular the generated sleef inline headers). We use
>> sleef api to vectorize
On Tue, 8 Oct 2024 15:13:06 GMT, Hamlin Li wrote:
>> Hamlin Li has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> keep comma
>
> Sorry for the inconvenience. Thanks for your quick re-reviews!
@Hamlin-Li are you still planning to re-submit
On Tue, 8 Oct 2024 14:56:56 GMT, Hamlin Li wrote:
>> Hi,
>> Can you help to review this patch?
>> Thanks!
>>
>> This patch is based on https://github.com/openjdk/jdk/pull/20781 which added
>> the sleef source (in particular the generated sleef inline headers). We use
>> sleef api to vectorize
On Tue, 8 Oct 2024 14:56:56 GMT, Hamlin Li wrote:
>> Hi,
>> Can you help to review this patch?
>> Thanks!
>>
>> This patch is based on https://github.com/openjdk/jdk/pull/20781 which added
>> the sleef source (in particular the generated sleef inline headers). We use
>> sleef api to vectorize
On Tue, 8 Oct 2024 14:56:56 GMT, Hamlin Li wrote:
>> Hi,
>> Can you help to review this patch?
>> Thanks!
>>
>> This patch is based on https://github.com/openjdk/jdk/pull/20781 which added
>> the sleef source (in particular the generated sleef inline headers). We use
>> sleef api to vectorize
On Tue, 8 Oct 2024 14:46:02 GMT, Magnus Ihse Bursie wrote:
>> Hamlin Li has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> make minor
>
> make/modules/jdk.incubator.vector/Lib.gmk line 53:
>
>> 51: DISABLED_WARNINGS_gcc := unused-fun
> Hi,
> Can you help to review this patch?
> Thanks!
>
> This patch is based on https://github.com/openjdk/jdk/pull/20781 which added
> the sleef source (in particular the generated sleef inline headers). We use
> sleef api to vectorize the math operations in vector api.
>
> On machine with vec
On Tue, 8 Oct 2024 14:07:45 GMT, Hamlin Li wrote:
>> Hi,
>> Can you help to review this patch?
>> Thanks!
>>
>> This patch is based on https://github.com/openjdk/jdk/pull/20781 which added
>> the sleef source (in particular the generated sleef inline headers). We use
>> sleef api to vectorize
> Hi,
> Can you help to review this patch?
> Thanks!
>
> This patch is based on https://github.com/openjdk/jdk/pull/20781 which added
> the sleef source (in particular the generated sleef inline headers). We use
> sleef api to vectorize the math operations in vector api.
>
> On machine with vec
On Mon, 7 Oct 2024 09:50:16 GMT, Hamlin Li wrote:
>> Hi,
>> Can you help to review this patch?
>> Thanks!
>>
>> This patch is based on https://github.com/openjdk/jdk/pull/20781 which added
>> the sleef source (in particular the generated sleef inline headers). We use
>> sleef api to vectorize
On Tue, 8 Oct 2024 13:40:53 GMT, Hamlin Li wrote:
>> So is the `LIBS` line as well. And the CFLAGS line should be just:
>>
>>CFLAGS := -march=rv64gcv, \
>
> No worries, it's not late. Take care!
Do you suggest something like below?
EXTRA_SRC := libsleef/generated, \
DISAB
On Tue, 8 Oct 2024 13:25:17 GMT, Magnus Ihse Bursie wrote:
>> make/modules/jdk.incubator.vector/Lib.gmk line 50:
>>
>>> 48: DISABLED_WARNINGS_clang := unused-function sign-compare
>>> tautological-compare ignored-qualifiers, \
>>> 49: CFLAGS := $(CFLAGS_JDKLIB) -march=rv64gcv, \
>>>
On Mon, 7 Oct 2024 09:50:16 GMT, Hamlin Li wrote:
>> Hi,
>> Can you help to review this patch?
>> Thanks!
>>
>> This patch is based on https://github.com/openjdk/jdk/pull/20781 which added
>> the sleef source (in particular the generated sleef inline headers). We use
>> sleef api to vectorize
On Mon, 7 Oct 2024 09:50:16 GMT, Hamlin Li wrote:
>> Hi,
>> Can you help to review this patch?
>> Thanks!
>>
>> This patch is based on https://github.com/openjdk/jdk/pull/20781 which added
>> the sleef source (in particular the generated sleef inline headers). We use
>> sleef api to vectorize
On Tue, 8 Oct 2024 13:23:21 GMT, Magnus Ihse Bursie wrote:
>> Hamlin Li has updated the pull request with a new target base due to a merge
>> or a rebase. The pull request now contains 11 commits:
>>
>> - merge master
>> - bits > 512
>> - fix test macro
>> - check frm after sleef call
>> -
On Mon, 7 Oct 2024 09:50:16 GMT, Hamlin Li wrote:
>> Hi,
>> Can you help to review this patch?
>> Thanks!
>>
>> This patch is based on https://github.com/openjdk/jdk/pull/20781 which added
>> the sleef source (in particular the generated sleef inline headers). We use
>> sleef api to vectorize
On Mon, 7 Oct 2024 09:16:42 GMT, Ludovic Henry wrote:
>> Hamlin Li has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> bits > 512
>
> Marked as reviewed by luhenry (Committer).
@luhenry I merged this pr with the master to resolve conflict a
On Mon, 23 Sep 2024 13:12:16 GMT, Erik Joelsson wrote:
>> Hamlin Li has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> modify cflags style
>
> Build changes look ok.
@erikj79 @RealFYang Hi, can you help to reapprove the pr? I guess it's be
On Mon, 7 Oct 2024 09:50:16 GMT, Hamlin Li wrote:
>> Hi,
>> Can you help to review this patch?
>> Thanks!
>>
>> This patch is based on https://github.com/openjdk/jdk/pull/20781 which added
>> the sleef source (in particular the generated sleef inline headers). We use
>> sleef api to vectorize
On Mon, 7 Oct 2024 09:50:16 GMT, Hamlin Li wrote:
>> Hi,
>> Can you help to review this patch?
>> Thanks!
>>
>> This patch is based on https://github.com/openjdk/jdk/pull/20781 which added
>> the sleef source (in particular the generated sleef inline headers). We use
>> sleef api to vectorize
> Hi,
> Can you help to review this patch?
> Thanks!
>
> This patch is based on https://github.com/openjdk/jdk/pull/20781 which added
> the sleef source (in particular the generated sleef inline headers). We use
> sleef api to vectorize the math operations in vector api.
>
> On machine with vec
On Mon, 30 Sep 2024 10:33:20 GMT, Hamlin Li wrote:
>> Hi,
>> Can you help to review this patch?
>> Thanks!
>>
>> This patch is based on https://github.com/openjdk/jdk/pull/20781 which added
>> the sleef source (in particular the generated sleef inline headers). We use
>> sleef api to vectorize
> Hi,
> Can you help to review this patch?
> Thanks!
>
> This patch is based on https://github.com/openjdk/jdk/pull/20781 which added
> the sleef source (in particular the generated sleef inline headers). We use
> sleef api to vectorize the math operations in vector api.
>
> On machine with vec
On Mon, 30 Sep 2024 09:01:24 GMT, Ludovic Henry wrote:
>> Hamlin Li has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> fix test macro
>
> src/hotspot/share/opto/vectorIntrinsics.cpp line 2044:
>
>> 2042: }
>> 2043:
>> 2044: if (addr =
On Thu, 26 Sep 2024 13:14:04 GMT, Hamlin Li wrote:
>> Hi,
>> Can you help to review this patch?
>> Thanks!
>>
>> This patch is based on https://github.com/openjdk/jdk/pull/20781 which added
>> the sleef source (in particular the generated sleef inline headers). We use
>> sleef api to vectorize
On Thu, 26 Sep 2024 13:14:04 GMT, Hamlin Li wrote:
>> Hi,
>> Can you help to review this patch?
>> Thanks!
>>
>> This patch is based on https://github.com/openjdk/jdk/pull/20781 which added
>> the sleef source (in particular the generated sleef inline headers). We use
>> sleef api to vectorize
On Fri, 27 Sep 2024 08:10:35 GMT, Fei Yang wrote:
>> `NDEBUG` is only used in sleefdp.c which is the original sleef code, and we
>> don't use that file in jdk directly, in java.base module of jdk it uses
>> `DEBUG` but not use `NDEBUG`.
>> Based on above information, I think `DEBUG` is better.
On Fri, 27 Sep 2024 07:06:44 GMT, Hamlin Li wrote:
>> src/jdk.incubator.vector/linux/native/libsleef/lib/vector_math_rvv.c line 51:
>>
>>> 49: // the dynamic rounding mode is always RNE.
>>> 50:
>>> 51: #ifdef DEBUG
>>
>> Question: Should we check for `NDEBUG` macro (A macro specified by C/C++
On Fri, 27 Sep 2024 02:22:58 GMT, Fei Yang wrote:
>> Hamlin Li has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> fix test macro
>
> src/jdk.incubator.vector/linux/native/libsleef/lib/vector_math_rvv.c line 51:
>
>> 49: // the dynamic roun
On Thu, 26 Sep 2024 13:14:04 GMT, Hamlin Li wrote:
>> Hi,
>> Can you help to review this patch?
>> Thanks!
>>
>> This patch is based on https://github.com/openjdk/jdk/pull/20781 which added
>> the sleef source (in particular the generated sleef inline headers). We use
>> sleef api to vectorize
> Hi,
> Can you help to review this patch?
> Thanks!
>
> This patch is based on https://github.com/openjdk/jdk/pull/20781 which added
> the sleef source (in particular the generated sleef inline headers). We use
> sleef api to vectorize the math operations in vector api.
>
> On machine with vec
On Thu, 26 Sep 2024 07:54:32 GMT, Hamlin Li wrote:
>> Just a bit worried about the fact that manipunating CSR could be very costly
>> on RISC-V. Another choice would be adding an assertion about FP rounding
>> mode expecting RNE when returning back from the SLEEF routine. I also
>> checked flo
> Hi,
> Can you help to review this patch?
> Thanks!
>
> This patch is based on https://github.com/openjdk/jdk/pull/20781 which added
> the sleef source (in particular the generated sleef inline headers). We use
> sleef api to vectorize the math operations in vector api.
>
> On machine with vec
On Thu, 26 Sep 2024 02:15:34 GMT, Fei Yang wrote:
>> I see sleef code only set frm to RNE, but I'm not quite sure.
>> Even if we can make sure current sleef only set frm to RNE, seems to me we
>> can not depends on current implement, it could change. Although good news is
>> we don't update sle
On Wed, 25 Sep 2024 20:10:41 GMT, Hamlin Li wrote:
>> Good question!
>> Let me do some further investigation and get back later.
>
> I see sleef code only set frm to RNE, but I'm not quite sure.
> Even if we can make sure current sleef only set frm to RNE, seems to me we
> can not depends on cur
On Wed, 25 Sep 2024 14:04:35 GMT, Hamlin Li wrote:
>> src/hotspot/cpu/riscv/riscv.ad line 10079:
>>
>>> 10077: match(CallLeafVector);
>>> 10078:
>>> 10079: effect(USE meth, KILL cr);
>>
>> I haven't checked the details of `CallLeafVector` yet. One more question
>> here. Is it safe to assu
On Wed, 25 Sep 2024 02:29:11 GMT, Fei Yang wrote:
>> Hamlin Li has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> use all arg v regs
>
> src/hotspot/cpu/riscv/riscv.ad line 10079:
>
>> 10077: match(CallLeafVector);
>> 10078:
>> 10079:
On Tue, 24 Sep 2024 19:12:53 GMT, Hamlin Li wrote:
>> Hi,
>> Can you help to review this patch?
>> Thanks!
>>
>> This patch is based on https://github.com/openjdk/jdk/pull/20781 which added
>> the sleef source (in particular the generated sleef inline headers). We use
>> sleef api to vectorize
On Tue, 24 Sep 2024 19:09:29 GMT, Hamlin Li wrote:
>> Then why would we put a constraint on the number of supported argument
>> vector registers here (v8-v15 instead of v8-v23)? Could we just support all
>> of them, i.e., v8-v23 to comply with the RISC-V psABI?
>
> There is no strong reason, ju
On Tue, 24 Sep 2024 15:12:29 GMT, Fei Yang wrote:
>> Yes, sorry what I meant, updated.
>
> Then why would we put a constraint on the number of supported argument vector
> registers here (v8-v15 instead of v8-v23)? Could we just support all of them,
> i.e., v8-v23 to comply with the RISC-V psABI
> Hi,
> Can you help to review this patch?
> Thanks!
>
> This patch is based on https://github.com/openjdk/jdk/pull/20781 which added
> the sleef source (in particular the generated sleef inline headers). We use
> sleef api to vectorize the math operations in vector api.
>
> On machine with vec
On Tue, 24 Sep 2024 14:57:48 GMT, Robbin Ehn wrote:
>> Thanks Robbin for helping explaining!
>>
>> minor correction: v1-v7/v24-v31 = callee saved
>
> Yes, sorry what I meant, updated.
Then why would we put a constraint on the number of supported argument vector
registers here (v8-v15 instead o
On Tue, 24 Sep 2024 14:52:39 GMT, Hamlin Li wrote:
>> Ah, I think I missed that. I was reading psABI spec 1.0 release. Thanks for
>> this info.
>
> Thanks Robbin for helping explaining!
>
> minor correction: v1-v7/v24-v31 = callee saved
Yes, sorry what I meant, updated.
-
PR Revi
On Tue, 24 Sep 2024 03:47:41 GMT, Fei Yang wrote:
>> Hamlin Li has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> refine comment
>
> src/hotspot/cpu/riscv/stubGenerator_riscv.cpp line 6063:
>
>> 6061:
>> 6062: void generate_vector_math_
On Tue, 24 Sep 2024 05:57:08 GMT, Robbin Ehn wrote:
>> src/hotspot/cpu/riscv/riscv.ad line 10078:
>>
>>> 10076: match(CallLeafVector);
>>> 10077:
>>> 10078: effect(USE meth);
>>
>> It's possible for the runtime call to clobber rFlagsReg `cr` (aka `t1`). So
>> safer to add `KILL cr` to the
On Tue, 24 Sep 2024 06:59:58 GMT, Fei Yang wrote:
>> Note in the RISC-V ELF psABI there is a convention variant for v-regs.
>> If you add function attribute riscv_vector_cc it should be used for C/C++.
>> (I never tested it)
>> v0 = first vector mask argument
>> v8-v23 = args/rets
>> v1-v7/v24-v
> Hi,
> Can you help to review this patch?
> Thanks!
>
> This patch is based on https://github.com/openjdk/jdk/pull/20781 which added
> the sleef source (in particular the generated sleef inline headers). We use
> sleef api to vectorize the math operations in vector api.
>
> On machine with vec
On Tue, 24 Sep 2024 05:25:38 GMT, Robbin Ehn wrote:
>> src/hotspot/cpu/riscv/assembler_riscv.hpp line 51:
>>
>>> 49: n_int_register_parameters_c = 8, // x10, x11, ... x17 (c_rarg0,
>>> c_rarg1, ...)
>>> 50: n_float_register_parameters_c = 8, // f10, f11, ... f17 (c_farg0,
>>> c_f
On Tue, 24 Sep 2024 03:50:29 GMT, Fei Yang wrote:
>> Hamlin Li has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> refine comment
>
> src/hotspot/cpu/riscv/riscv.ad line 10078:
>
>> 10076: match(CallLeafVector);
>> 10077:
>> 10078: eff
On Tue, 24 Sep 2024 03:45:32 GMT, Fei Yang wrote:
>> Hamlin Li has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> refine comment
>
> src/hotspot/cpu/riscv/assembler_riscv.hpp line 51:
>
>> 49: n_int_register_parameters_c = 8, // x1
On Mon, 23 Sep 2024 19:32:10 GMT, Hamlin Li wrote:
>> Hi,
>> Can you help to review this patch?
>> Thanks!
>>
>> This patch is based on https://github.com/openjdk/jdk/pull/20781 which added
>> the sleef source (in particular the generated sleef inline headers). We use
>> sleef api to vectorize
On Mon, 23 Sep 2024 16:40:55 GMT, Ludovic Henry wrote:
>> Hamlin Li has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> comment
>
> src/jdk.incubator.vector/linux/native/libsleef/lib/vector_math_rvv.c line 31:
>
>> 29: // bridge functions b
> Hi,
> Can you help to review this patch?
> Thanks!
>
> This patch is based on https://github.com/openjdk/jdk/pull/20781 which added
> the sleef source (in particular the generated sleef inline headers). We use
> sleef api to vectorize the math operations in vector api.
>
> On machine with vec
On Mon, 23 Sep 2024 16:25:02 GMT, Hamlin Li wrote:
>> Hi,
>> Can you help to review this patch?
>> Thanks!
>>
>> This patch is based on https://github.com/openjdk/jdk/pull/20781 which added
>> the sleef source (in particular the generated sleef inline headers). We use
>> sleef api to vectorize
On Mon, 23 Sep 2024 09:09:46 GMT, Ludovic Henry wrote:
>> Hamlin Li has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> modify cflags style
>
> src/jdk.incubator.vector/linux/native/libsleef/lib/vector_math_rvv.c line 24:
>
>> 22: */
>> 23
On Mon, 23 Sep 2024 07:30:59 GMT, Hamlin Li wrote:
>> Hi,
>> Can you help to review this patch?
>> Thanks!
>>
>> This patch is based on https://github.com/openjdk/jdk/pull/20781 which added
>> the sleef source (in particular the generated sleef inline headers). We use
>> sleef api to vectorize
> Hi,
> Can you help to review this patch?
> Thanks!
>
> This patch is based on https://github.com/openjdk/jdk/pull/20781 which added
> the sleef source (in particular the generated sleef inline headers). We use
> sleef api to vectorize the math operations in vector api.
>
> On machine with vec
On Mon, 23 Sep 2024 07:30:59 GMT, Hamlin Li wrote:
>> Hi,
>> Can you help to review this patch?
>> Thanks!
>>
>> This patch is based on https://github.com/openjdk/jdk/pull/20781 which added
>> the sleef source (in particular the generated sleef inline headers). We use
>> sleef api to vectorize
On Mon, 23 Sep 2024 07:30:59 GMT, Hamlin Li wrote:
>> Hi,
>> Can you help to review this patch?
>> Thanks!
>>
>> This patch is based on https://github.com/openjdk/jdk/pull/20781 which added
>> the sleef source (in particular the generated sleef inline headers). We use
>> sleef api to vectorize
On Fri, 20 Sep 2024 12:34:38 GMT, Erik Joelsson wrote:
>> Thanks. I'm sorry too, I'm not familiar with the build system.
>> What you expected could be something like below?
>>
>> diff --git a/make/modules/jdk.incubator.vector/Lib.gmk
>> b/make/modules/jdk.incubator.vector/Lib.gmk
>> index 5e522
> Hi,
> Can you help to review this patch?
> Thanks!
>
> This patch is based on https://github.com/openjdk/jdk/pull/20781 which added
> the sleef source (in particular the generated sleef inline headers). We use
> sleef api to vectorize the math operations in vector api.
>
> On machine with vec
On Thu, 19 Sep 2024 16:35:14 GMT, Hamlin Li wrote:
>> Sorry, I had to remind myself of how this works. We actually set this as a
>> separate parameter on the Setup macro: `OPTIMIZATION := HIGH`
>
> Thanks. I'm sorry too, I'm not familiar with the build system.
> What you expected could be someth
On Thu, 19 Sep 2024 08:32:38 GMT, Hamlin Li wrote:
> Hi,
> Can you help to review this patch?
> Thanks!
>
> This patch is based on https://github.com/openjdk/jdk/pull/20781 which added
> the sleef source (in particular the generated sleef inline headers). We use
> sleef api to vectorize the ma
On Thu, 19 Sep 2024 16:02:36 GMT, Erik Joelsson wrote:
>> Thanks, do you mean something like below? I'll fix it.
>>
>> CFLAGS := $(CFLAGS_JDKLIB) $(C_O_FLAG_HI) -march=rv64gcv, \
>
> Sorry, I had to remind myself of how this works. We actually set this as a
> separate parameter on the Setup mac
On Thu, 19 Sep 2024 13:47:50 GMT, Hamlin Li wrote:
>> make/modules/jdk.incubator.vector/Lib.gmk line 48:
>>
>>> 46: DISABLED_WARNINGS_gcc := unused-function sign-compare
>>> tautological-compare ignored-qualifiers, \
>>> 47: DISABLED_WARNINGS_clang := unused-function sign-compare
>
On Thu, 19 Sep 2024 12:52:59 GMT, Erik Joelsson wrote:
>> Hi,
>> Can you help to review this patch?
>> Thanks!
>>
>> This patch is based on https://github.com/openjdk/jdk/pull/20781 which added
>> the sleef source (in particular the generated sleef inline headers). We use
>> sleef api to vecto
On Thu, 19 Sep 2024 08:32:38 GMT, Hamlin Li wrote:
> Hi,
> Can you help to review this patch?
> Thanks!
>
> This patch is based on https://github.com/openjdk/jdk/pull/20781 which added
> the sleef source (in particular the generated sleef inline headers). We use
> sleef api to vectorize the ma
On Thu, 19 Sep 2024 08:32:38 GMT, Hamlin Li wrote:
> Hi,
> Can you help to review this patch?
> Thanks!
>
> This patch is based on https://github.com/openjdk/jdk/pull/20781 which added
> the sleef source (in particular the generated sleef inline headers). We use
> sleef api to vectorize the ma
Hi,
Can you help to review this patch?
Thanks!
This patch is based on https://github.com/openjdk/jdk/pull/20781 which added
the sleef source (in particular the generated sleef inline headers). We use
sleef api to vectorize the math operations in vector api.
On machine with vector intrinsic supp
69 matches
Mail list logo