commit a231ea6dcbee90c0079cad1645783f0311a02a6c
author Georgios Pinitas <georgios.pinitas@arm.com> Tue Jul 30 12:21:01 2019 +0100
committer Michele Di Giorgio <michele.digiorgio@arm.com> Wed Jul 31 14:39:03 2019 +0000
tree d7832f294b061eefc3200c971617a88ba974ea05
parent 5d42b4635f60b57a81b702d924a6e9b5b7cfbc01

COMPMID-2336: Fix GEMMLowp 32bit check

Fix invalid check for performing fused output stage in the intrinsic
fallback path of NEGEMMLowpCore

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

src/runtime/NEON/functions/NEGEMMLowpMatrixMultiplyCore.cpp

diff --git a/src/runtime/NEON/functions/NEGEMMLowpMatrixMultiplyCore.cpp b/src/runtime/NEON/functions/NEGEMMLowpMatrixMultiplyCore.cpp
index 6dc5dd2..a03ec10 100644
--- a/src/runtime/NEON/functions/NEGEMMLowpMatrixMultiplyCore.cpp
+++ b/src/runtime/NEON/functions/NEGEMMLowpMatrixMultiplyCore.cpp
@@ -242,7 +242,7 @@
     int32_t a_offset = a->quantization_info().uniform().offset;
     int32_t b_offset = b->quantization_info().uniform().offset;
 
-    bool fuse_output_stage = gemm_info.gemmlowp_output_stage().type != GEMMLowpOutputStageType::NONE && a->data_type() != DataType::QASYMM8;
+    bool fuse_output_stage = gemm_info.gemmlowp_output_stage().type != GEMMLowpOutputStageType::NONE;
     if(fuse_output_stage)
     {
         auto_init_if_empty(mm_result_s32_info, a->clone()->set_tensor_shape(output->tensor_shape()).set_data_type(DataType::S32));