diff options
| author | Gian Marco Iodice <gianmarco.iodice@arm.com> | 2021-09-22 15:38:01 +0100 |
|---|---|---|
| committer | Giorgio Arena <giorgio.arena@arm.com> | 2021-09-23 09:26:17 +0000 |
| commit | 0bb62303c76cf90570fbaee9a03f65f408127ae8 (patch) | |
| tree | 9070231834cda2c8d26dd7657bca178019b7e312 | |
| parent | 732d8798ee83dc871c5468d4faaf63554678a119 (diff) | |
| download | ComputeLibrary-0bb62303c76cf90570fbaee9a03f65f408127ae8.tar.gz | |
Fix inefficient store in gemmlowp_mm_reshaped_only_rhs_t
- The out-of-boundary condition was performed also for PARTIAL_STORE_N0
= 0
Resolves: COMPMID-4774 COMPMID-4771
Change-Id: I0d7e078c67615b513ffeb66860f224999b5135fa
Signed-off-by: Gian Marco Iodice <gianmarco.iodice@arm.com>
Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/6302
Tested-by: Arm Jenkins <bsgcomp@arm.com>
Reviewed-by: Giorgio Arena <giorgio.arena@arm.com>
| -rw-r--r-- | src/core/CL/cl_kernels/common/gemmlowp.cl | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/core/CL/cl_kernels/common/gemmlowp.cl b/src/core/CL/cl_kernels/common/gemmlowp.cl index 5cafb5389c..564cbf93cc 100644 --- a/src/core/CL/cl_kernels/common/gemmlowp.cl +++ b/src/core/CL/cl_kernels/common/gemmlowp.cl @@ -786,7 +786,7 @@ __kernel void gemmlowp_mm_reshaped_only_rhs_t #endif // defined(REINTERPRET_OUTPUT_AS_3D) }) - const bool cond_x = (xo > (N - N0)); + const bool cond_x = (xo > (N - N0)) && (PARTIAL_STORE_N0 != 0); #if defined(FUSED_OUTPUT_STAGE_FIXED_POINT) T_STORE_INDIRECT_WIDTH_SELECT(DATA_TYPE, M0, N0, PARTIAL_STORE_N0, BUFFER, dst, xo, dst_stride_y, cond_x, c_lp, dst_indirect_y); |