commit 8a1ac6162e2ed07384f5a51162e5c5e347a88c67
author Georgios Pinitas <georgios.pinitas@arm.com> Thu Apr 08 12:25:36 2021 +0100
committer Michele Di Giorgio <michele.digiorgio@arm.com> Thu Apr 08 17:28:15 2021 +0000
tree 99424da10ab04603817b195d7c13f03ce6fb5540
parent 4a5f73d36962f254ad630beb70deb1907ba381b9

Fix incorrect return statement in gemm_uint8 heuristic selection

Semantic fix that otherwise led to compilation errors when building for
SVE and when MMLA instruction was enabled for int8.

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

src/core/NEON/kernels/arm_gemm/gemm_uint8.cpp

diff --git a/src/core/NEON/kernels/arm_gemm/gemm_uint8.cpp b/src/core/NEON/kernels/arm_gemm/gemm_uint8.cpp
index fb41a9f..4de3d2b 100644
--- a/src/core/NEON/kernels/arm_gemm/gemm_uint8.cpp
+++ b/src/core/NEON/kernels/arm_gemm/gemm_uint8.cpp
@@ -51,7 +51,7 @@
 {
     GemmMethod::GEMM_INTERLEAVED,
     "sve_interleaved_u8u32_mmla_8x3VL",
-    [](const GemmArgs &args) {  args._ci->has_sve() && return (args._Ksize>8); },
+    [](const GemmArgs &args) { return args._ci->has_sve() && (args._Ksize>8); },
     [](const GemmArgs &args) { return args._ci->get_cpu_model() != CPUModel::KLEIN; },
     [](const GemmArgs &args) { return new GemmInterleaved<cls_sve_interleaved_u8u32_mmla_8x3VL, uint8_t, uint32_t>(args); }
 },