From 0a99c7953027c17d686f8f012495a7e4d8a969cf Mon Sep 17 00:00:00 2001 From: Adnan AlSinan Date: Tue, 3 Oct 2023 15:15:41 +0100 Subject: Fix nightly NEON Reverse reference failure - Fix the reference axis vector to be the right size. - Update typos in the error messages. Resolves COMPMID-6574 Signed-off-by: Adnan AlSinan Change-Id: I9572365b8173b92d0fffd557e4db261b2969109c Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/10423 Tested-by: Arm Jenkins Benchmark: Arm Jenkins Reviewed-by: SiCong Li --- src/core/NEON/kernels/NEReverseKernel.cpp | 2 +- tests/validation/fixtures/ReverseFixture.h | 2 +- tests/validation/reference/Reverse.cpp | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/core/NEON/kernels/NEReverseKernel.cpp b/src/core/NEON/kernels/NEReverseKernel.cpp index d2437eecd0..b3710555df 100644 --- a/src/core/NEON/kernels/NEReverseKernel.cpp +++ b/src/core/NEON/kernels/NEReverseKernel.cpp @@ -106,7 +106,7 @@ void run_reverse( // The values of axis tensor must be between [-rank, rank-1]. if ((axis_i < -rank) || (axis_i >= rank)) { - ARM_COMPUTE_ERROR("the valuses of the axis tensor must be within [-rank, rank-1]."); + ARM_COMPUTE_ERROR("the values of the axis tensor must be within [-rank, rank-1]."); } // In case of negative axis value i.e targeted axis(i) = rank + axis(i) diff --git a/tests/validation/fixtures/ReverseFixture.h b/tests/validation/fixtures/ReverseFixture.h index 7518a0926a..3bf4c3e327 100644 --- a/tests/validation/fixtures/ReverseFixture.h +++ b/tests/validation/fixtures/ReverseFixture.h @@ -152,7 +152,7 @@ protected: // Fill reference fill(src); auto axis_v = generate_random_axis(use_negative_axis); - std::copy(axis_v.begin(), axis_v.begin() + axis_shape.x(), axis.data()); + std::copy(axis_v.begin(), axis_v.begin() + _num_dims, axis.data()); return reference::reverse(src, axis, use_inverted_axis); } diff --git a/tests/validation/reference/Reverse.cpp b/tests/validation/reference/Reverse.cpp index 5fd15b5bfc..7924f900d1 100644 --- a/tests/validation/reference/Reverse.cpp +++ b/tests/validation/reference/Reverse.cpp @@ -59,7 +59,7 @@ SimpleTensor reverse(const SimpleTensor &src, const SimpleTensor // The values of axis tensor must be between [-rank, rank-1]. if((axis_i < -rank) || (axis_i >= rank)) { - ARM_COMPUTE_ERROR("the valuses of the axis tensor must be within [-rank, rank-1]."); + ARM_COMPUTE_ERROR("the values of the axis tensor must be within [-rank, rank-1]."); } // In case of negative axis value i.e targeted axis(i) = rank + axis(i) -- cgit v1.2.1