From 9d0b5f82c2734444145718f12788f2dde436ef45 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: I78039db8c58d7b14a042c41e54c25fb9cb509bf7 Signed-off-by: Michalis Spyrou Reviewed-on: https://review.mlplatform.org/c/1092 Reviewed-by: VidhyaSudhan Loganathan Comments-Addressed: Arm Jenkins Tested-by: Arm Jenkins --- arm_compute/core/GLES_COMPUTE/GCKernelLibrary.h | 4 ++-- arm_compute/core/NEON/kernels/NECropKernel.h | 6 +++--- arm_compute/core/utils/logging/LoggerRegistry.h | 2 +- 3 files changed, 6 insertions(+), 6 deletions(-) (limited to 'arm_compute/core') diff --git a/arm_compute/core/GLES_COMPUTE/GCKernelLibrary.h b/arm_compute/core/GLES_COMPUTE/GCKernelLibrary.h index c0d64e2edb..89d6d17eb5 100644 --- a/arm_compute/core/GLES_COMPUTE/GCKernelLibrary.h +++ b/arm_compute/core/GLES_COMPUTE/GCKernelLibrary.h @@ -1,5 +1,5 @@ /* - * Copyright (c) 2017-2018 ARM Limited. + * Copyright (c) 2017-2019 ARM Limited. * * SPDX-License-Identifier: MIT * @@ -271,7 +271,7 @@ private: * * @return Preprocessed GLES shader object. */ - const std::string preprocess_shader(const std::string &shader_source) const; + std::string preprocess_shader(const std::string &shader_source) const; /** Load program and its dependencies. * * @param[in] program_name Name of the program to load. diff --git a/arm_compute/core/NEON/kernels/NECropKernel.h b/arm_compute/core/NEON/kernels/NECropKernel.h index 6713a40c86..ef89d112f6 100644 --- a/arm_compute/core/NEON/kernels/NECropKernel.h +++ b/arm_compute/core/NEON/kernels/NECropKernel.h @@ -107,14 +107,14 @@ private: uint32_t _crop_box_ind; float _extrapolation_value; /** The number of rows out of bounds at the start and end of output. */ - uint32_t _rows_out_of_bounds[2]; + std::array _rows_out_of_bounds; /** The number of columns out of bounds at the start and end of output. */ - uint32_t _cols_out_of_bounds[2]; + std::array _cols_out_of_bounds; std::pair _in_bounds_crop_functions; NECropKernel::InBoundsCropFunction *_in_bounds_crop_function; - using CropFunction = void(const ITensor *, const ITensor *, Coordinates, float, const uint32_t *, const uint32_t *, + using CropFunction = void(const ITensor *, const ITensor *, Coordinates, float, const std::array &, const std::array &, NECropKernel::InBoundsCropFunction *); NECropKernel::CropFunction *_crop_function; diff --git a/arm_compute/core/utils/logging/LoggerRegistry.h b/arm_compute/core/utils/logging/LoggerRegistry.h index c841020c22..0e223ff663 100644 --- a/arm_compute/core/utils/logging/LoggerRegistry.h +++ b/arm_compute/core/utils/logging/LoggerRegistry.h @@ -55,7 +55,7 @@ public: * @param[in] printers Printers to attach to the system loggers. Defaults with a @ref StdPrinter. */ void create_logger(const std::string &name, LogLevel log_level = LogLevel::INFO, - std::vector> printers = { std::make_shared() }); + const std::vector> &printers = { std::make_shared() }); /** Remove a logger * * @param name Logger's name -- cgit v1.2.1