RFR: 8350463: AArch64: Add vector rearrange support for small lane count vectors

Xiaohong Gong xgong at openjdk.org
Thu Mar 20 07:31:09 UTC 2025


On Tue, 18 Mar 2025 02:36:44 GMT, Xiaohong Gong <xgong at openjdk.org> wrote:

>>> Thanks for looking at this PR again @eme64 ! Vector API has its own jtreg tests under `test/jdk/jdk/incubator/vector/`. I double checked that it has the `rearrange` test for all vector species. Please see one of the test here: https://github.com/openjdk/jdk/blob/master/test/jdk/jdk/incubator/vector/Long128VectorTests.java#L4954 That's also way I did not add the correct tests in the IR test file.
>> 
>> Alright. I think result verification would still be good practice, and not too difficult to do using a `@Check` method and `Verify.java` for comparing the resulting arrays. But I leave that up to you. In my experience, the VectorAPI test coverage is not as good as I first thought, see the list of bugs I recently found:
>> https://bugs.openjdk.org/issues/?jql=labels%20%3D%20template-framework%20ORDER%20BY%20created%20DESC%2C%20summary%20DESC
>> 
>> So adding a little more rigor to your IR test could catch possible bugs that the existing tests simply do not cover.
>
> Hi @eme64 , the IR test is updated according to your suggestion. Could you please look at it again? Thanks so much!

> @XiaohongGong Could you please also merge here before I rerun the testing?

Sure and have rebased. Thanks a lot for your testing!

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

PR Comment: https://git.openjdk.org/jdk/pull/23790#issuecomment-2739452172


More information about the hotspot-compiler-dev mailing list