Skip to content

Commit

Permalink
Fix qdmlsl instructions
Browse files Browse the repository at this point in the history
The qdmlsl instructions were implemented without any saturation.
This has been fixed by utilising existing saturating instructions which
are implemented correctly.

Unit tests have also been updated to test for saturation.
  • Loading branch information
Ryo-not-rio authored and mr-c committed Nov 25, 2024
1 parent b285190 commit facf2cb
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/arm/neon/qdmlsl_high_n.c
Original file line number Diff line number Diff line change
Expand Up @@ -115,9 +115,9 @@ test_simde_vqdmlsl_high_n_s32 (SIMDE_MUNIT_TEST_ARGS) {
INT32_C( 1),
{ INT64_MAX, INT64_MIN } },
{ { INT64_C( 0), INT64_C( 0) },
{ INT32_C( INT32_MIN), INT32_C( 97715), -INT32_C( 1), INT32_C( 0) },
{ INT32_C( INT32_MIN), INT32_C( 97715), -INT32_C( INT32_MIN), INT32_C( 0) },
INT32_C( INT32_MIN),
{ INT64_C( INT64_MIN + 1), INT64_C( 0) } },
{ INT64_MIN + 1, INT64_C( 0) } },
};

for (size_t i = 0 ; i < (sizeof(test_vec) / sizeof(test_vec[0])) ; i++) {
Expand Down

0 comments on commit facf2cb

Please sign in to comment.