Skip to content

Add tests for the NEON simulator.#1067

Merged
mhucka merged 2 commits into
mainfrom
add-neon-tests
May 16, 2026
Merged

Add tests for the NEON simulator.#1067
mhucka merged 2 commits into
mainfrom
add-neon-tests

Conversation

@sergeisakov
Copy link
Copy Markdown
Collaborator

This pull request adds tests for the NEON statespace and simulator.

@github-actions github-actions Bot added the size: L 250< lines changed <1000 label May 13, 2026
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request adds support for NEON instructions to the build system and introduces new test files for SimulatorNEON and StateSpaceNEON. The reviewer recommends replacing typedef with using for type aliases and updating them to follow the CamelCase naming convention in accordance with the Google C++ Style Guide.

Comment thread tests/simulator_neon_test.cc
Comment thread tests/simulator_neon_test.cc
Comment thread tests/statespace_neon_test.cc
Comment thread tests/statespace_neon_test.cc
@sergeisakov sergeisakov requested a review from mhucka May 13, 2026 13:34
Copy link
Copy Markdown
Collaborator

@mhucka mhucka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@mhucka mhucka added this pull request to the merge queue May 16, 2026
Merged via the queue into main with commit 3643d80 May 16, 2026
33 checks passed
@mhucka mhucka deleted the add-neon-tests branch May 16, 2026 02:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: L 250< lines changed <1000

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants