diff options
author | Giorgio Arena <giorgio.arena@arm.com> | 2021-08-20 15:24:03 +0100 |
---|---|---|
committer | Giorgio Arena <giorgio.arena@arm.com> | 2021-08-24 11:46:13 +0000 |
commit | 9c67d3883adaa8a65e3567579cf99937deaab21a (patch) | |
tree | e3b11c5918b4121e8e7d5ad728847ce74a8ed77f /src/graph/Tensor.cpp | |
parent | 87a74effff65f6fa1b0e565818e02c3b414ae1cf (diff) | |
download | ComputeLibrary-9c67d3883adaa8a65e3567579cf99937deaab21a.tar.gz |
Remove map/unmap overhead for input/output accessor when using DummyAccessor
Don't map/unmap when the tensor data is not accessed in the input or output accessor.
This is so to avoid measuring CPU overhead when benchmarking on the GPU backend.
Resolve COMPMID-4712
Change-Id: I5baba1b93e7a51fe13525bcce6c0cfdecb14493e
Signed-off-by: Giorgio Arena <giorgio.arena@arm.com>
Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/6140
Comments-Addressed: Arm Jenkins <bsgcomp@arm.com>
Reviewed-by: Georgios Pinitas <georgios.pinitas@arm.com>
Tested-by: Arm Jenkins <bsgcomp@arm.com>
Diffstat (limited to 'src/graph/Tensor.cpp')
-rw-r--r-- | src/graph/Tensor.cpp | 24 |
1 files changed, 16 insertions, 8 deletions
diff --git a/src/graph/Tensor.cpp b/src/graph/Tensor.cpp index f69d49d016..3d4723430f 100644 --- a/src/graph/Tensor.cpp +++ b/src/graph/Tensor.cpp @@ -1,5 +1,5 @@ /* - * Copyright (c) 2018-2019 Arm Limited. + * Copyright (c) 2018-2019,2021 Arm Limited. * * SPDX-License-Identifier: MIT * @@ -80,20 +80,28 @@ bool Tensor::call_accessor() return false; } - // Map tensor - _handle->map(true); + const bool access_data = _accessor->access_tensor_data(); - // Return in case of null backend buffer - if(_handle->tensor().buffer() == nullptr) + if(access_data) { - return false; + // Map tensor + _handle->map(true); + + // Return in case of null backend buffer + if(_handle->tensor().buffer() == nullptr) + { + return false; + } } // Call accessor bool retval = _accessor->access_tensor(_handle->tensor()); - // Unmap tensor - _handle->unmap(); + if(access_data) + { + // Unmap tensor + _handle->unmap(); + } return retval; } |