From 980558373f14c135a331fa23b61558c7d177edf2 Mon Sep 17 00:00:00 2001 From: Georgios Pinitas Date: Tue, 27 Jul 2021 10:34:59 +0100 Subject: Fix memory lifetime issue B matrix was getting released while was still needed for a Native GEMM execution Resolves: COMPMID-4705 Signed-off-by: Georgios Pinitas Change-Id: I1e490abbe1feabd451d10fa785de21e725f1d6e0 Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/6001 Comments-Addressed: Arm Jenkins --- src/runtime/gpu/cl/operators/ClGemmLowpMatrixMultiplyCore.cpp | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) (limited to 'src/runtime/gpu') diff --git a/src/runtime/gpu/cl/operators/ClGemmLowpMatrixMultiplyCore.cpp b/src/runtime/gpu/cl/operators/ClGemmLowpMatrixMultiplyCore.cpp index 82047ad2a5..64c8743f13 100644 --- a/src/runtime/gpu/cl/operators/ClGemmLowpMatrixMultiplyCore.cpp +++ b/src/runtime/gpu/cl/operators/ClGemmLowpMatrixMultiplyCore.cpp @@ -335,8 +335,6 @@ void ClGemmLowpMatrixMultiplyCore::configure(const CLCompileContext &compile_con { _run_output_stage = true; - // _memory_group.manage(&_mm_result_s32); - if(_is_gemm_reshaped) { _mm_reshaped_only_rhs_kernel->configure(compile_context, a, matrix_b, &_mm_result_s32, gemm_kernel_info); @@ -726,12 +724,11 @@ void ClGemmLowpMatrixMultiplyCore::prepare(ITensorPack &tensors) { ITensorPack convert_to_qs8_pack = { { ACL_SRC, b }, { ACL_DST, rhs_qasymm8.get() } }; CLScheduler::get().enqueue_op(*_weights_to_qasymm8, convert_to_qs8_pack, false); + b->mark_as_unused(); } if(_is_gemm_reshaped && _reshape_b_only_on_first_run) { - ARM_COMPUTE_ERROR_ON(!b->is_used()); - // Run reshape kernel and mark original weights tensor as unused ITensorPack mtx_b_pack = { -- cgit v1.2.1