COMPMID-2401: Fix CLGemmLowp macro expansion on no-dot platforms
Change-Id: If707865ff13c96627816863cd05e09aaef247bbe
Signed-off-by: Georgios Pinitas <georgios.pinitas@arm.com>
Reviewed-on: https://review.mlplatform.org/c/1361
Reviewed-by: Michele Di Giorgio <michele.digiorgio@arm.com>
Reviewed-by: Michalis Spyrou <michalis.spyrou@arm.com>
Comments-Addressed: Arm Jenkins <bsgcomp@arm.com>
Tested-by: Arm Jenkins <bsgcomp@arm.com>
diff --git a/src/core/CL/cl_kernels/gemmlowp.cl b/src/core/CL/cl_kernels/gemmlowp.cl
index 54ffea1..65c31ef 100644
--- a/src/core/CL/cl_kernels/gemmlowp.cl
+++ b/src/core/CL/cl_kernels/gemmlowp.cl
@@ -66,13 +66,13 @@
#else // defined(ARM_COMPUTE_OPENCL_DOT8_ENABLED) && defined(cl_arm_integer_dot_product_int8)
/** Specialized macros to perform the dot product instruction between two vectors of size K0 [1,16] without using the dot8 instruction. */
-#define ARM_DOT1(a, b, c) \
- ({ \
- c += (uint)a.s0 * b.s0; \
+#define ARM_DOT1(a, b, c) \
+ ({ \
+ c += (uint)a * b; \
})
#define ARM_DOT2(a, b, c) \
({ \
- ARM_DOT1(a, b, c); \
+ c += (uint)a.s0 * b.s0; \
c += (uint)a.s1 * b.s1; \
})
#define ARM_DOT3(a, b, c) \
@@ -134,7 +134,7 @@
ARM_DOT_K0(k0, (a), (b##F), (c.sF)); \
})
-/** Specialized macros to perform a a partial matrix multiplication with dimensions M0,N0,K0*/
+/** Specialized macros to perform a a partial matrix multiplication with dimensions M0,N0,K0 */
#define ARM_MM_K0XN0X1(n0, k0, a, b, c) \
({ \
ARM_DOT_K0XN0(n0, k0, (a##0), b, (c##0)); \