Skip to content

tests: extend the SIMD kernel equivalence tests to NEON and SVE - #40

Merged
chishui merged 1 commit into
opensearch-project:mainfrom
zirui-song-18:arm-kernel-tests
Aug 31, 2026
Merged

chishui merged 1 commit into
opensearch-project:mainfrom
zirui-song-18:arm-kernel-tests

Conversation

@zirui-song-18

Copy link
Copy Markdown
Collaborator

Description

Extends the per-ISA kernel-equivalence tests (#39) to ARM. NEON links the base nsparse (the aarch64 baseline) and runs ungated; SVE links nsparse_sve, compiles with -march=armv8-a+sve, is built only off Apple, and is launch-gated on the sve CPU flag (a fully-+sve binary can execute SVE in a static initializer, so it must not start on a non-SVE CPU). NEON is exercised by CI's macOS (Apple Silicon) runner; SVE runs where the host advertises it (Graviton3).

Tested SVE on a r7g instance:

Start 607: nsparse_neon_kernel_test
607/608 Test #607: nsparse_neon_kernel_test ........................................................................................................   Passed    0.00 sec
        Start 608: nsparse_sve_kernel_test
608/608 Test #608: nsparse_sve_kernel_test .........................................................................................................   Passed    0.01 sec

100% tests passed, 0 tests failed out of 608

Total Test time (real) =   6.40 sec

Issues Resolved

List any issues this PR will resolve, e.g. Closes [...].

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Zirui Song <zrsong@amazon.com>
@chishui
chishui merged commit 4082c24 into opensearch-project:main Aug 31, 2026
9 checks passed
@chishui chishui mentioned this pull request Sep 2, 2026
3 tasks done
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants