diff options
author | SiCong Li <sicong.li@arm.com> | 2022-11-09 16:01:44 +0000 |
---|---|---|
committer | SiCong Li <sicong.li@arm.com> | 2022-11-10 17:24:16 +0000 |
commit | fd76611f50b0abe5effbe4b7f7129a745115fdf9 (patch) | |
tree | 79290d0b9c914a122fa9b299b97e72e0240ac046 /src/dynamic_fusion | |
parent | d4a9cc00a666c7d4c2a35c49d71b322f27e369fc (diff) | |
download | ComputeLibrary-fd76611f50b0abe5effbe4b7f7129a745115fdf9.tar.gz |
Fix compiler warnings in dynamic fusion
Remove conflicting Padding2D from the unused comparison operator in the
prototpye
Resolve unused variables in release mode
Resolves COMPMID-5683
Signed-off-by: SiCong Li <sicong.li@arm.com>
Change-Id: I19d74c57e51e6cf64003ddcbc74227608bb866d2
Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/8590
Reviewed-by: Jakub Sujak <jakub.sujak@arm.com>
Reviewed-by: Viet-Hoa Do <viet-hoa.do@arm.com>
Tested-by: Arm Jenkins <bsgcomp@arm.com>
Benchmark: Arm Jenkins <bsgcomp@arm.com>
Comments-Addressed: Arm Jenkins <bsgcomp@arm.com>
Diffstat (limited to 'src/dynamic_fusion')
-rw-r--r-- | src/dynamic_fusion/sketch/gpu/GpuKernelComponentGraph.h | 1 | ||||
-rw-r--r-- | src/dynamic_fusion/sketch/gpu/GpuLogicalKernel.cpp | 1 | ||||
-rw-r--r-- | src/dynamic_fusion/sketch/utils/DependencyGraph.h | 1 |
3 files changed, 3 insertions, 0 deletions
diff --git a/src/dynamic_fusion/sketch/gpu/GpuKernelComponentGraph.h b/src/dynamic_fusion/sketch/gpu/GpuKernelComponentGraph.h index fbcb2c10ea..e4f498b130 100644 --- a/src/dynamic_fusion/sketch/gpu/GpuKernelComponentGraph.h +++ b/src/dynamic_fusion/sketch/gpu/GpuKernelComponentGraph.h @@ -76,6 +76,7 @@ public: const auto src_tensor_ids = get_tensor_ids(tensors.get_const_src_tensors()); const auto dst_tensor_ids = get_tensor_ids(tensors.get_const_dst_tensors()); bool success = _dependency_graph.add_operator(comp->id(), src_tensor_ids, dst_tensor_ids); + ARM_COMPUTE_UNUSED(success); ARM_COMPUTE_ERROR_ON(!success); _components[comp->id()] = std::move(comp); for(auto t : tensors.get_const_src_tensors()) diff --git a/src/dynamic_fusion/sketch/gpu/GpuLogicalKernel.cpp b/src/dynamic_fusion/sketch/gpu/GpuLogicalKernel.cpp index 7746f8bbf3..c560e9a931 100644 --- a/src/dynamic_fusion/sketch/gpu/GpuLogicalKernel.cpp +++ b/src/dynamic_fusion/sketch/gpu/GpuLogicalKernel.cpp @@ -79,6 +79,7 @@ void GpuLogicalKernel::add_load_store() tensors); _store_components.push_back(std::move(store)); auto success = _comp_group.add_component(_store_components.back().get()); + ARM_COMPUTE_UNUSED(success); ARM_COMPUTE_ERROR_ON(!success); // It's guaranteed that any load store insertion should be successful } } diff --git a/src/dynamic_fusion/sketch/utils/DependencyGraph.h b/src/dynamic_fusion/sketch/utils/DependencyGraph.h index 55eb4c5c77..03678defae 100644 --- a/src/dynamic_fusion/sketch/utils/DependencyGraph.h +++ b/src/dynamic_fusion/sketch/utils/DependencyGraph.h @@ -130,6 +130,7 @@ public: { ARM_COMPUTE_ERROR_ON(!try_add_operator_as_linear(op, inputs, outputs)); auto success = add_operator(op, inputs, outputs); + ARM_COMPUTE_UNUSED(success); ARM_COMPUTE_ERROR_ON(!success); } /** Add a new operator |