Re: RFR: 8355563: VectorAPI: Refactor current implementation of subword gather load API [v2]

2025-07-06 Thread Xiaohong Gong
On Mon, 7 Jul 2025 02:05:06 GMT, Xiaohong Gong wrote: >> @XiaohongGong I quickly scanned the patch, it looks good to me too. I'm >> submitting some internal testing now, to make sure our extended testing does >> not break on integration. Should take about 24h. > > Hi @eme64 , may I ask how the

Re: RFR: 8355563: VectorAPI: Refactor current implementation of subword gather load API [v2]

2025-07-06 Thread Emanuel Peter
On Wed, 25 Jun 2025 09:16:48 GMT, Xiaohong Gong wrote: >> JDK-8318650 introduced hotspot intrinsification of subword gather load APIs >> for X86 platforms [1]. However, the current implementation is not optimal >> for AArch64 SVE platform, which natively supports vector instructions for >> sub

Re: RFR: 8355563: VectorAPI: Refactor current implementation of subword gather load API [v2]

2025-07-06 Thread Xiaohong Gong
On Wed, 2 Jul 2025 08:24:22 GMT, Emanuel Peter wrote: >>> Agree with Paul, these are minor regressions. Let us proceed with this >>> patch. >> >> Thanks so much for your review @sviswa7 ! > > @XiaohongGong I quickly scanned the patch, it looks good to me too. I'm > submitting some internal tes

Re: RFR: 8355563: VectorAPI: Refactor current implementation of subword gather load API [v2]

2025-07-02 Thread Xiaohong Gong
On Wed, 2 Jul 2025 01:52:19 GMT, Xiaohong Gong wrote: >> Agree with Paul, these are minor regressions. Let us proceed with this >> patch. > >> Agree with Paul, these are minor regressions. Let us proceed with this patch. > > Thanks so much for your review @sviswa7 ! > @XiaohongGong I quickly

Re: RFR: 8355563: VectorAPI: Refactor current implementation of subword gather load API [v2]

2025-07-02 Thread Emanuel Peter
On Wed, 2 Jul 2025 01:52:19 GMT, Xiaohong Gong wrote: >> Agree with Paul, these are minor regressions. Let us proceed with this >> patch. > >> Agree with Paul, these are minor regressions. Let us proceed with this patch. > > Thanks so much for your review @sviswa7 ! @XiaohongGong I quickly sc

Re: RFR: 8355563: VectorAPI: Refactor current implementation of subword gather load API [v2]

2025-07-01 Thread Xiaohong Gong
On Tue, 1 Jul 2025 21:30:20 GMT, Sandhya Viswanathan wrote: > Agree with Paul, these are minor regressions. Let us proceed with this patch. Thanks so much for your review @sviswa7 ! - PR Comment: https://git.openjdk.org/jdk/pull/25138#issuecomment-3026080679

Re: RFR: 8355563: VectorAPI: Refactor current implementation of subword gather load API [v2]

2025-07-01 Thread Xiaohong Gong
On Tue, 1 Jul 2025 18:03:33 GMT, Paul Sandoz wrote: > This is a nice simplification, Java changes look good. I'll let the Intel > folks sign-off related to regressions. IMO minor regressions like this are > acceptable if the generated code quality is good, and if the benchmark > reports higher

Re: RFR: 8355563: VectorAPI: Refactor current implementation of subword gather load API [v2]

2025-07-01 Thread Sandhya Viswanathan
On Wed, 25 Jun 2025 09:16:48 GMT, Xiaohong Gong wrote: >> JDK-8318650 introduced hotspot intrinsification of subword gather load APIs >> for X86 platforms [1]. However, the current implementation is not optimal >> for AArch64 SVE platform, which natively supports vector instructions for >> sub

Re: RFR: 8355563: VectorAPI: Refactor current implementation of subword gather load API [v2]

2025-07-01 Thread Paul Sandoz
On Wed, 25 Jun 2025 09:16:48 GMT, Xiaohong Gong wrote: >> JDK-8318650 introduced hotspot intrinsification of subword gather load APIs >> for X86 platforms [1]. However, the current implementation is not optimal >> for AArch64 SVE platform, which natively supports vector instructions for >> sub

Re: RFR: 8355563: VectorAPI: Refactor current implementation of subword gather load API [v2]

2025-07-01 Thread Xiaohong Gong
On Wed, 25 Jun 2025 09:16:48 GMT, Xiaohong Gong wrote: >> JDK-8318650 introduced hotspot intrinsification of subword gather load APIs >> for X86 platforms [1]. However, the current implementation is not optimal >> for AArch64 SVE platform, which natively supports vector instructions for >> sub

Re: RFR: 8355563: VectorAPI: Refactor current implementation of subword gather load API [v2]

2025-07-01 Thread Xiaohong Gong
On Tue, 1 Jul 2025 06:41:32 GMT, Xiaohong Gong wrote: >> Ping again! Thanks in advance! > >> @XiaohongGong I'm a little busy at the moment, and soon going on a summer >> vacation, so I cannot promise a full review soon. Feel free to ask someone >> else to have a look. >> >> I quickly looked th

Re: RFR: 8355563: VectorAPI: Refactor current implementation of subword gather load API [v2]

2025-07-01 Thread Xiaohong Gong
On Tue, 1 Jul 2025 06:07:03 GMT, Xiaohong Gong wrote: >> Xiaohong Gong has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains three commits: >> >> - Address review comments >> - Merge 'jdk:master' into JDK-8355563 >> - 8355563: Vector

Re: RFR: 8355563: VectorAPI: Refactor current implementation of subword gather load API [v2]

2025-06-30 Thread Emanuel Peter
On Tue, 1 Jul 2025 06:41:32 GMT, Xiaohong Gong wrote: >> Ping again! Thanks in advance! > >> @XiaohongGong I'm a little busy at the moment, and soon going on a summer >> vacation, so I cannot promise a full review soon. Feel free to ask someone >> else to have a look. >> >> I quickly looked th

Re: RFR: 8355563: VectorAPI: Refactor current implementation of subword gather load API [v2]

2025-06-30 Thread Emanuel Peter
On Tue, 1 Jul 2025 06:07:03 GMT, Xiaohong Gong wrote: >> Xiaohong Gong has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains three commits: >> >> - Address review comments >> - Merge 'jdk:master' into JDK-8355563 >> - 8355563: Vector

Re: RFR: 8355563: VectorAPI: Refactor current implementation of subword gather load API

2025-06-25 Thread Xiaohong Gong
On Fri, 9 May 2025 07:35:41 GMT, Xiaohong Gong wrote: > JDK-8318650 introduced hotspot intrinsification of subword gather load APIs > for X86 platforms [1]. However, the current implementation is not optimal for > AArch64 SVE platform, which natively supports vector instructions for subword >

Re: RFR: 8355563: VectorAPI: Refactor current implementation of subword gather load API [v2]

2025-06-25 Thread Xiaohong Gong
> JDK-8318650 introduced hotspot intrinsification of subword gather load APIs > for X86 platforms [1]. However, the current implementation is not optimal for > AArch64 SVE platform, which natively supports vector instructions for subword > gather load operations using an int vector for indices (

Re: RFR: 8355563: VectorAPI: Refactor current implementation of subword gather load API

2025-06-25 Thread Xiaohong Gong
On Mon, 2 Jun 2025 10:48:25 GMT, Emanuel Peter wrote: >>> > @XiaohongGong Thanks for splitting this one out, and for investigating >>> > the regressions here. >>> > Putting the permalink here, fixed to the current change (the link you >>> > pasted will always refer to the newest, which may late

Re: RFR: 8355563: VectorAPI: Refactor current implementation of subword gather load API

2025-06-02 Thread Xiaohong Gong
On Tue, 3 Jun 2025 01:45:57 GMT, Xiaohong Gong wrote: >>> > @XiaohongGong Thanks for splitting this one out, and for investigating >>> > the regressions here. >>> > Putting the permalink here, fixed to the current change (the link you >>> > pasted will always refer to the newest, which may late

Re: RFR: 8355563: VectorAPI: Refactor current implementation of subword gather load API

2025-06-02 Thread Xiaohong Gong
On Fri, 30 May 2025 08:15:22 GMT, Xiaohong Gong wrote: >>> @XiaohongGong Thanks for splitting this one out, and for investigating the >>> regressions here. >>> >>> Putting the permalink here, fixed to the current change (the link you >>> pasted will always refer to the newest, which may later

Re: RFR: 8355563: VectorAPI: Refactor current implementation of subword gather load API

2025-06-02 Thread Emanuel Peter
On Fri, 30 May 2025 08:15:22 GMT, Xiaohong Gong wrote: >>> @XiaohongGong Thanks for splitting this one out, and for investigating the >>> regressions here. >>> >>> Putting the permalink here, fixed to the current change (the link you >>> pasted will always refer to the newest, which may later

Re: RFR: 8355563: VectorAPI: Refactor current implementation of subword gather load API

2025-06-02 Thread Jatin Bhateja
On Fri, 9 May 2025 07:35:41 GMT, Xiaohong Gong wrote: > JDK-8318650 introduced hotspot intrinsification of subword gather load APIs > for X86 platforms [1]. However, the current implementation is not optimal for > AArch64 SVE platform, which natively supports vector instructions for subword >

Re: RFR: 8355563: VectorAPI: Refactor current implementation of subword gather load API

2025-05-30 Thread Xiaohong Gong
On Tue, 20 May 2025 05:40:04 GMT, Xiaohong Gong wrote: > > @XiaohongGong Thanks for splitting this one out, and for investigating the > > regressions here. > > Putting the permalink here, fixed to the current change (the link you > > pasted will always refer to the newest, which may later on po

Re: RFR: 8355563: VectorAPI: Refactor current implementation of subword gather load API

2025-05-26 Thread Xiaohong Gong
On Mon, 26 May 2025 06:51:12 GMT, Emanuel Peter wrote: >> JDK-8318650 introduced hotspot intrinsification of subword gather load APIs >> for X86 platforms [1]. However, the current implementation is not optimal >> for AArch64 SVE platform, which natively supports vector instructions for >> sub

Re: RFR: 8355563: VectorAPI: Refactor current implementation of subword gather load API

2025-05-25 Thread Emanuel Peter
On Fri, 9 May 2025 07:35:41 GMT, Xiaohong Gong wrote: > JDK-8318650 introduced hotspot intrinsification of subword gather load APIs > for X86 platforms [1]. However, the current implementation is not optimal for > AArch64 SVE platform, which natively supports vector instructions for subword >

Re: RFR: 8355563: VectorAPI: Refactor current implementation of subword gather load API

2025-05-19 Thread Xiaohong Gong
On Tue, 20 May 2025 02:22:13 GMT, Xiaohong Gong wrote: >> Ping again~ could any one please take a look at this PR? Thanks a lot! > >> Hi @XiaohongGong , Very nice work!, Looks good to me, will do some testing >> and get back. >> >> Do you have any idea about following regression? >> >> ``` >>

Re: RFR: 8355563: VectorAPI: Refactor current implementation of subword gather load API

2025-05-19 Thread Emanuel Peter
On Tue, 20 May 2025 02:22:13 GMT, Xiaohong Gong wrote: >> Ping again~ could any one please take a look at this PR? Thanks a lot! > >> Hi @XiaohongGong , Very nice work!, Looks good to me, will do some testing >> and get back. >> >> Do you have any idea about following regression? >> >> ``` >>

Re: RFR: 8355563: VectorAPI: Refactor current implementation of subword gather load API

2025-05-19 Thread Xiaohong Gong
On Mon, 19 May 2025 03:10:46 GMT, Xiaohong Gong wrote: >> JDK-8318650 introduced hotspot intrinsification of subword gather load APIs >> for X86 platforms [1]. However, the current implementation is not optimal >> for AArch64 SVE platform, which natively supports vector instructions for >> sub

Re: RFR: 8355563: VectorAPI: Refactor current implementation of subword gather load API

2025-05-19 Thread Jatin Bhateja
On Mon, 19 May 2025 03:10:46 GMT, Xiaohong Gong wrote: >> JDK-8318650 introduced hotspot intrinsification of subword gather load APIs >> for X86 platforms [1]. However, the current implementation is not optimal >> for AArch64 SVE platform, which natively supports vector instructions for >> sub

Re: RFR: 8355563: VectorAPI: Refactor current implementation of subword gather load API

2025-05-18 Thread Xiaohong Gong
On Fri, 9 May 2025 07:35:41 GMT, Xiaohong Gong wrote: > JDK-8318650 introduced hotspot intrinsification of subword gather load APIs > for X86 platforms [1]. However, the current implementation is not optimal for > AArch64 SVE platform, which natively supports vector instructions for subword >

Re: RFR: 8355563: VectorAPI: Refactor current implementation of subword gather load API

2025-05-13 Thread Xiaohong Gong
On Fri, 9 May 2025 07:35:41 GMT, Xiaohong Gong wrote: > JDK-8318650 introduced hotspot intrinsification of subword gather load APIs > for X86 platforms [1]. However, the current implementation is not optimal for > AArch64 SVE platform, which natively supports vector instructions for subword >

Re: RFR: 8355563: VectorAPI: Refactor current implementation of subword gather load API

2025-05-09 Thread Xiaohong Gong
On Fri, 9 May 2025 07:35:41 GMT, Xiaohong Gong wrote: > JDK-8318650 introduced hotspot intrinsification of subword gather load APIs > for X86 platforms [1]. However, the current implementation is not optimal for > AArch64 SVE platform, which natively supports vector instructions for subword >

RFR: 8355563: VectorAPI: Refactor current implementation of subword gather load API

2025-05-09 Thread Xiaohong Gong
JDK-8318650 introduced hotspot intrinsification of subword gather load APIs for X86 platforms [1]. However, the current implementation is not optimal for AArch64 SVE platform, which natively supports vector instructions for subword gather load operations using an int vector for indices (see [2][