commit 63d4dbddf9d7b6cdd0e682ae838dc3ef37eb95ce
author Georgios Pinitas <georgios.pinitas@arm.com> Fri Nov 08 11:51:56 2019 +0000
committer Georgios Pinitas <georgios.pinitas@arm.com> Fri Nov 08 15:21:44 2019 +0000
tree d72936e2dcd267cdc241e75597a23492612d3189
parent 15396ff07911fb806b468962f746c38d206bac7e

COMPMID-2452: Fix 32-bit per-channel convolution for NEON.

Rearrange the kernels in run to ensure type conversion takes place
before the matrix transformations.

Change-Id: Ibf47788fe71a84fd7549f8667549552e15ca8aab
Signed-off-by: Georgios Pinitas <georgios.pinitas@arm.com>
Reviewed-on: https://review.mlplatform.org/c/2251
Reviewed-by: Michele Di Giorgio <michele.digiorgio@arm.com>
Comments-Addressed: Arm Jenkins <bsgcomp@arm.com>
Tested-by: Arm Jenkins <bsgcomp@arm.com>

src/core/NEON/kernels/NEGEMMLowpMatrixMultiplyKernel.cpp

diff --git a/src/core/NEON/kernels/NEGEMMLowpMatrixMultiplyKernel.cpp b/src/core/NEON/kernels/NEGEMMLowpMatrixMultiplyKernel.cpp
index 8f5a208..3082ff2 100644
--- a/src/core/NEON/kernels/NEGEMMLowpMatrixMultiplyKernel.cpp
+++ b/src/core/NEON/kernels/NEGEMMLowpMatrixMultiplyKernel.cpp
@@ -870,6 +870,7 @@
         switch(_input0->info()->data_type())
         {
             case DataType::S8:
+            case DataType::QASYMM8_SIGNED:
             {
                 vector_matrix_multiply_s8(ina, inb, out, width_matrix_a, width_matrix_b, in_b_stride, window);
                 break;