tree eeee19f204dfcac7808fd71a683a57b5e231d964
parent b526431b1079a86b30a1bf6e905435e7121584f9
author Gunes Bayir <gunes.bayir@arm.com> 1702460287 +0000
committer Gunes Bayir <gunes.bayir@arm.com> 1702465315 +0000

Fix nightly bug caused by wrong validation in Gemm mmul kernel

While writing this gemm kernel, code pieces, including validations were adapted from ClGemmMatrixMultiplyReshapedOnlyRhsKernel, and this validation should be about reinterpret_input_as_3d, which is not dealt with in this kernel. The mmul kernel only deals with reinterpret_output_as_3d, which is equivalent to depth_output_gemm3d != 0. This reveals a test gap for this kernel. There are currently no tests stressing this condition; but this is not going to be addressed as part of the bug ticket.

The corresponding snippet in ClGemmMatrixMultiplyReshapedOnlyRhsKernel is

if (gemm_info.reinterpret_input_as_3d)
{
        ARM_COMPUTE_RETURN_ERROR_ON(src0->dimension(1) * src0->dimension(2) != m);
 }
 else
 {
        ARM_COMPUTE_RETURN_ERROR_ON(src0->dimension(1) != m);
 }

Resolves: COMPMID-6757

Change-Id: I73b203594b22098a5374c1fac6969ee769969901
Signed-off-by: Gunes Bayir <gunes.bayir@arm.com>
Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/10874
Benchmark: Arm Jenkins <bsgcomp@arm.com>
Reviewed-by: Jakub Sujak <jakub.sujak@arm.com>
Comments-Addressed: Arm Jenkins <bsgcomp@arm.com>
Tested-by: Arm Jenkins <bsgcomp@arm.com>