From 299fdd31bd8e1add3ac557a5e630de55b1b6659c Mon Sep 17 00:00:00 2001 From: Michalis Spyrou Date: Wed, 1 May 2019 13:03:59 +0100 Subject: COMPMID-2177 Fix clang warnings Change-Id: I4beacfd714ee3ed771fd174cce5d8009a2961380 Signed-off-by: Michalis Spyrou Reviewed-on: https://review.mlplatform.org/c/1065 Reviewed-by: Giuseppe Rossini Tested-by: Arm Jenkins --- src/runtime/NEON/functions/NEDepthConcatenateLayer.cpp | 18 +++++++++++------- 1 file changed, 11 insertions(+), 7 deletions(-) (limited to 'src/runtime/NEON/functions/NEDepthConcatenateLayer.cpp') diff --git a/src/runtime/NEON/functions/NEDepthConcatenateLayer.cpp b/src/runtime/NEON/functions/NEDepthConcatenateLayer.cpp index b814bffa96..8f070a2d7d 100644 --- a/src/runtime/NEON/functions/NEDepthConcatenateLayer.cpp +++ b/src/runtime/NEON/functions/NEDepthConcatenateLayer.cpp @@ -45,9 +45,7 @@ NEDepthConcatenateLayer::NEDepthConcatenateLayer() // NOLINT void NEDepthConcatenateLayer::configure(const std::vector &inputs_vector, ITensor *output) // NOLINT { - _num_inputs = inputs_vector.size(); - _concat_kernels_vector = arm_compute::support::cpp14::make_unique(_num_inputs); - _border_handlers_vector = arm_compute::support::cpp14::make_unique(_num_inputs); + _num_inputs = inputs_vector.size(); std::vector inputs_vector_info; for(unsigned int i = 0; i < _num_inputs; i++) @@ -61,10 +59,16 @@ void NEDepthConcatenateLayer::configure(const std::vector &inputs_vec ARM_COMPUTE_ERROR_THROW_ON(NEDepthConcatenateLayer::validate(inputs_vector_info, output->info())); unsigned int depth_offset = 0; + _concat_kernels_vector.reserve(_num_inputs); + _border_handlers_vector.reserve(_num_inputs); for(unsigned int i = 0; i < _num_inputs; ++i) { - _concat_kernels_vector[i].configure(inputs_vector.at(i), depth_offset, output); - _border_handlers_vector[i].configure(inputs_vector.at(i), _concat_kernels_vector[i].border_size(), BorderMode::CONSTANT, PixelValue(static_cast(0.f))); + auto concat_kernel = support::cpp14::make_unique(); + auto border_kernel = support::cpp14::make_unique(); + concat_kernel->configure(inputs_vector.at(i), depth_offset, output); + border_kernel->configure(inputs_vector.at(i), concat_kernel->border_size(), BorderMode::CONSTANT, PixelValue(static_cast(0.f))); + _border_handlers_vector.emplace_back(std::move(border_kernel)); + _concat_kernels_vector.emplace_back(std::move(concat_kernel)); depth_offset += inputs_vector.at(i)->info()->dimension(2); } @@ -98,7 +102,7 @@ void NEDepthConcatenateLayer::run() { for(unsigned i = 0; i < _num_inputs; ++i) { - NEScheduler::get().schedule(&_border_handlers_vector[i], Window::DimX); - NEScheduler::get().schedule(&_concat_kernels_vector[i], Window::DimX); + NEScheduler::get().schedule(_border_handlers_vector[i].get(), Window::DimX); + NEScheduler::get().schedule(_concat_kernels_vector[i].get(), Window::DimX); } } -- cgit v1.2.1