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

[Xiaohong Gong]

On Tue, 20 May 2025 05:40:04 GMT, Xiaohong Gong <xgong at openjdk.org> 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 point to the wrong line when lines above are inserted / deleted):
> > https://github.com/openjdk/jdk/blob/7077535c0b0a6ea0a2a167f9135b1504a3d71fb3/src/hotspot/share/opto/loopnode.cpp#L1659-L1661
> > 
> > I wonder if we should just use `Node::uncast` there? But I'm quite unsure about that.
> 
> Sounds good to me. I will have a deep investigation for it. Thanks!

Hi @eme64  @jatin-bhateja,  I'v created a PR https://github.com/openjdk/jdk/pull/25539 to fix this issue. With this change, the performance regression can be fixed as well.  Could you please take a look at that change and help to run the test on different X86 machines?   Thanks a lot!

-------------

PR Comment: https://git.openjdk.org/jdk/pull/25138#issuecomment-2921574716