diff options
author | Michalis Spyrou <michalis.spyrou@arm.com> | 2020-05-06 17:03:59 +0100 |
---|---|---|
committer | Michalis Spyrou <michalis.spyrou@arm.com> | 2020-05-20 21:48:34 +0000 |
commit | 18e20ff8bc31bd835e96b46c0beb5435c76572bd (patch) | |
tree | fcd826b6f528da07d4b82d0300a19f78a6ac7333 /src | |
parent | 11bfe0e99b3af0edbed1bfd2b89242b3b4e45d72 (diff) | |
download | ComputeLibrary-18e20ff8bc31bd835e96b46c0beb5435c76572bd.tar.gz |
COMPMID-3460: Refactor NEElementwiseUnaryKernel
Removed most of the templates and refactored the code.
Performance is the same but the libary size dropped by
52Kb.
Change-Id: I41ff0c0853c923d925cdaeb05f4a58c9086fff94
Signed-off-by: Michalis Spyrou <michalis.spyrou@arm.com>
Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/3190
Tested-by: Arm Jenkins <bsgcomp@arm.com>
Reviewed-by: Michele Di Giorgio <michele.digiorgio@arm.com>
Comments-Addressed: Arm Jenkins <bsgcomp@arm.com>
Diffstat (limited to 'src')
-rw-r--r-- | src/core/NEON/kernels/NEElementwiseUnaryKernel.cpp | 142 |
1 files changed, 34 insertions, 108 deletions
diff --git a/src/core/NEON/kernels/NEElementwiseUnaryKernel.cpp b/src/core/NEON/kernels/NEElementwiseUnaryKernel.cpp index 5d3af3b03d..ba93c37ba1 100644 --- a/src/core/NEON/kernels/NEElementwiseUnaryKernel.cpp +++ b/src/core/NEON/kernels/NEElementwiseUnaryKernel.cpp @@ -1,5 +1,5 @@ /* - * Copyright (c) 2018-2019 ARM Limited. + * Copyright (c) 2018-2020 ARM Limited. * * SPDX-License-Identifier: MIT * @@ -26,30 +26,17 @@ #include "arm_compute/core/CPP/Validate.h" #include "arm_compute/core/Error.h" #include "arm_compute/core/Helpers.h" -#include "arm_compute/core/IAccessWindow.h" #include "arm_compute/core/ITensor.h" -#include "arm_compute/core/NEON/NEAsymm.h" -#include "arm_compute/core/NEON/NEFixedPoint.h" -#include "arm_compute/core/NEON/NEMath.h" #include "arm_compute/core/NEON/wrapper/wrapper.h" -#include "arm_compute/core/TensorInfo.h" #include "arm_compute/core/Validate.h" #include "support/ToolchainSupport.h" -#include <algorithm> -#include <arm_neon.h> -#include <cstdint> -#include <map> -#include <string> - namespace arm_compute { -class Coordinates; - namespace { -template <ElementWiseUnary op, typename ScalarType> -inline ScalarType elementwise_op_scalar(const ScalarType &a) +template <typename ScalarType> +inline ScalarType elementwise_op_scalar_imp(ElementWiseUnary op, const ScalarType &a) { switch(op) { @@ -72,9 +59,8 @@ inline ScalarType elementwise_op_scalar(const ScalarType &a) } } -/* Elementwise operations that are supported for float */ -template <ElementWiseUnary op, typename ScalarType, bool is_float, typename VectorType, typename std::enable_if<is_float, int>::type = 0> -inline VectorType elementwise_op(const VectorType &a) +template <typename ScalarType, typename VectorType> +inline VectorType elementwise_op_imp(ElementWiseUnary op, const VectorType &a) { switch(op) { @@ -96,24 +82,10 @@ inline VectorType elementwise_op(const VectorType &a) ARM_COMPUTE_ERROR("NOT_SUPPORTED!"); } } +} // namespace -/* Elementwise operations that are supported for non floats */ -template < ElementWiseUnary op, typename ScalarType, bool is_float, typename VectorType, typename std::enable_if < !is_float, int >::type = 0 > -inline VectorType elementwise_op(const VectorType &a) -{ - switch(op) - { - case ElementWiseUnary::NEG: - return wrapper::vneg(a); - case ElementWiseUnary::ABS: - return wrapper::vabs(a); - default: - ARM_COMPUTE_ERROR("NOT_SUPPORTED!"); - } -} - -template <ElementWiseUnary op, typename ScalarType, bool is_float> -void elementwise_op(const ITensor *in, ITensor *out, const Window &window) +template <typename ScalarType> +void NEElementwiseUnaryKernel::elementwise_op(const Window &window) { const int window_step_x = 16 / sizeof(ScalarType); const auto window_start_x = static_cast<int>(window.x().start()); @@ -122,8 +94,8 @@ void elementwise_op(const ITensor *in, ITensor *out, const Window &window) Window win = window; win.set(Window::DimX, Window::Dimension(0, 1, 1)); - Iterator input(in, win); - Iterator output(out, win); + Iterator input(_input, win); + Iterator output(_output, win); execute_window_loop(win, [&](const Coordinates &) { @@ -133,55 +105,24 @@ void elementwise_op(const ITensor *in, ITensor *out, const Window &window) int x = window_start_x; for(; x <= window_end_x - window_step_x; x += window_step_x) { - wrapper::vstore(output_ptr + x, elementwise_op<op, ScalarType, is_float>(wrapper::vloadq(input_ptr + x))); + wrapper::vstore(output_ptr + x, elementwise_op_imp<ScalarType>(_op, wrapper::vloadq(input_ptr + x))); } for(; x < window_end_x; ++x) { - *(output_ptr + x) = elementwise_op_scalar<op>(*(input_ptr + x)); + *(output_ptr + x) = elementwise_op_scalar_imp(_op, *(input_ptr + x)); } }, input, output); } -template <ElementWiseUnary op> -std::function<void(const ITensor *input, ITensor *output, const Window &window)> -configure_func(const ITensor *input, ITensor *output) -{ - std::string function_to_call("op_"); - function_to_call += string_from_data_type(input->info()->data_type()) + "_"; - function_to_call += string_from_data_type(output->info()->data_type()); - - static std::map<std::string, NEElementwiseUnaryKernel::ElementwiseUnaryFunction *> map_function = - { - { "op_F32_F32", &elementwise_op<op, float, true> }, - { "op_S32_S32", &elementwise_op<op, int32_t, false> }, - }; -#ifdef __ARM_FEATURE_FP16_VECTOR_ARITHMETIC - map_function["op_F16_F16"] = &elementwise_op<op, float16_t, true>; -#endif /* __ARM_FEATURE_FP16_VECTOR_ARITHMETIC */ - - auto it = map_function.find(function_to_call); - - if(it != map_function.end()) - { - auto func = it->second; - return [func](const ITensor * input, ITensor * output, const Window & window) - { - func(input, output, window); - }; - } - return nullptr; -} -} // namespace - NEElementwiseUnaryKernel::NEElementwiseUnaryKernel() - : _function(nullptr), _input(nullptr), _output(nullptr) + : _func(nullptr), _input(nullptr), _output(nullptr), _op() { } void NEElementwiseUnaryKernel::configure(ElementWiseUnary op, const ITensor *input, ITensor *output) { - ARM_COMPUTE_ERROR_THROW_ON(validate_arguments(op, *input->info(), *output->info())); + ARM_COMPUTE_ERROR_THROW_ON(validate(op, input->info(), output->info())); ARM_COMPUTE_ERROR_ON_NULLPTR(input, output); // Configure kernel window @@ -196,40 +137,32 @@ void NEElementwiseUnaryKernel::configure(ElementWiseUnary op, const ITensor *inp _input = input; _output = output; + _op = op; INEKernel::configure(win); - switch(op) + switch(input->info()->data_type()) { - case ElementWiseUnary::RSQRT: - _function = configure_func<ElementWiseUnary::RSQRT>(input, output); + case DataType::F32: + _func = &NEElementwiseUnaryKernel::elementwise_op<float>; break; - case ElementWiseUnary::EXP: - _function = configure_func<ElementWiseUnary::EXP>(input, output); - break; - case ElementWiseUnary::NEG: - _function = configure_func<ElementWiseUnary::NEG>(input, output); - break; - case ElementWiseUnary::LOG: - _function = configure_func<ElementWiseUnary::LOG>(input, output); - break; - case ElementWiseUnary::ABS: - _function = configure_func<ElementWiseUnary::ABS>(input, output); - break; - case ElementWiseUnary::ROUND: - _function = configure_func<ElementWiseUnary::ROUND>(input, output); +#ifdef __ARM_FEATURE_FP16_VECTOR_ARITHMETIC + case DataType::F16: + _func = &NEElementwiseUnaryKernel::elementwise_op<float16_t>; +#endif /* __ARM_FEATURE_FP16_VECTOR_ARITHMETIC */ break; - case ElementWiseUnary::SIN: - _function = configure_func<ElementWiseUnary::SIN>(input, output); + case DataType::S32: + _func = &NEElementwiseUnaryKernel::elementwise_op<int32_t>; break; default: - ARM_COMPUTE_ERROR("NOT_SUPPORTED!"); + ARM_COMPUTE_ERROR("DataType not supported"); } } -Status NEElementwiseUnaryKernel::validate_arguments(ElementWiseUnary op, const ITensorInfo &input, const ITensorInfo &output) +Status NEElementwiseUnaryKernel::validate(ElementWiseUnary op, const ITensorInfo *input, const ITensorInfo *output) { - ARM_COMPUTE_RETURN_ERROR_ON_CPU_F16_UNSUPPORTED(&input); + ARM_COMPUTE_RETURN_ERROR_ON_NULLPTR(input, output); + ARM_COMPUTE_RETURN_ERROR_ON_CPU_F16_UNSUPPORTED(input); switch(op) { case ElementWiseUnary::EXP: @@ -237,37 +170,30 @@ Status NEElementwiseUnaryKernel::validate_arguments(ElementWiseUnary op, const I case ElementWiseUnary::LOG: case ElementWiseUnary::ROUND: case ElementWiseUnary::SIN: - ARM_COMPUTE_RETURN_ERROR_ON_DATA_TYPE_CHANNEL_NOT_IN(&input, 1, DataType::F16, DataType::F32); + ARM_COMPUTE_RETURN_ERROR_ON_DATA_TYPE_CHANNEL_NOT_IN(input, 1, DataType::F16, DataType::F32); break; case ElementWiseUnary::NEG: case ElementWiseUnary::ABS: - ARM_COMPUTE_RETURN_ERROR_ON_DATA_TYPE_CHANNEL_NOT_IN(&input, 1, DataType::F16, DataType::F32, DataType::S32); + ARM_COMPUTE_RETURN_ERROR_ON_DATA_TYPE_CHANNEL_NOT_IN(input, 1, DataType::F16, DataType::F32, DataType::S32); break; default: ARM_COMPUTE_ERROR("ElementWiseUnary operation not supported"); } // Validate in case of configured output - if(output.total_size() > 0) + if(output->total_size() > 0) { - ARM_COMPUTE_RETURN_ERROR_ON_MISMATCHING_DATA_TYPES(&input, &output); + ARM_COMPUTE_RETURN_ERROR_ON_MISMATCHING_DATA_TYPES(input, output); } return Status{}; } -Status NEElementwiseUnaryKernel::validate(ElementWiseUnary op, const ITensorInfo *input, const ITensorInfo *output) -{ - ARM_COMPUTE_RETURN_ERROR_ON_NULLPTR(input, output); - ARM_COMPUTE_RETURN_ON_ERROR(validate_arguments(op, *input, *output)); - return Status{}; -} - void NEElementwiseUnaryKernel::run(const Window &window, const ThreadInfo &info) { ARM_COMPUTE_UNUSED(info); ARM_COMPUTE_ERROR_ON_UNCONFIGURED_KERNEL(this); ARM_COMPUTE_ERROR_ON_INVALID_SUBWINDOW(INEKernel::window(), window); - ARM_COMPUTE_ERROR_ON(_function == nullptr); - _function(_input, _output, window); + ARM_COMPUTE_ERROR_ON(_func == nullptr); + (this->*_func)(window); } } // namespace arm_compute |