diff options
| author | Gian Marco Iodice <gianmarco.iodice@arm.com> | 2021-08-11 14:06:28 +0100 |
|---|---|---|
| committer | SiCongLi <sicong.li@arm.com> | 2021-08-11 16:19:48 +0100 |
| commit | 593c2425e6b94828fb486244e42c275a89a71aff (patch) | |
| tree | 27046940a8ff7faa33374982f695d216212840c1 | |
| parent | e63b41b56235cdcc19c4d76156bfd1b16b98012e (diff) | |
| download | ComputeLibrary-593c2425e6b94828fb486244e42c275a89a71aff.tar.gz | |
Fix performance regression due to clFinish()
- In ClGemmLowpMatrixMultiplyCore::prepare we always called clFinish()
also when the workload was already prepared
Resolves COMPMID-4707
Change-Id: Icdcee528590e2c5efb75325a80c2a45ec84993d1
Signed-off-by: Gian Marco Iodice <gianmarco.iodice@arm.com>
Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/6082
Tested-by: Arm Jenkins <bsgcomp@arm.com>
Reviewed-by: Giorgio Arena <giorgio.arena@arm.com>
| -rw-r--r-- | src/runtime/gpu/cl/operators/ClGemmLowpMatrixMultiplyCore.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/runtime/gpu/cl/operators/ClGemmLowpMatrixMultiplyCore.cpp b/src/runtime/gpu/cl/operators/ClGemmLowpMatrixMultiplyCore.cpp index 64c8743f13..0c72912642 100644 --- a/src/runtime/gpu/cl/operators/ClGemmLowpMatrixMultiplyCore.cpp +++ b/src/runtime/gpu/cl/operators/ClGemmLowpMatrixMultiplyCore.cpp @@ -773,9 +773,9 @@ void ClGemmLowpMatrixMultiplyCore::prepare(ITensorPack &tensors) shifts_tensor->unmap(CLScheduler::get().queue()); } } + CLScheduler::get().queue().finish(); _is_prepared = true; } - CLScheduler::get().queue().finish(); } experimental::MemoryRequirements ClGemmLowpMatrixMultiplyCore::workspace() const |