diff options
author | Usama Arif <usama.arif@arm.com> | 2019-05-14 10:22:36 +0100 |
---|---|---|
committer | Usama Arif <usama.arif@arm.com> | 2019-05-15 17:09:25 +0000 |
commit | 52c54f61b97bcedab309bfa761e193939e12e739 (patch) | |
tree | e99186abb3c6a69ef26b17e8fb57817ee56efe53 /src/core | |
parent | eb312ef6c20e8548c43eb9d4a3edf7265bc6777b (diff) | |
download | ComputeLibrary-52c54f61b97bcedab309bfa761e193939e12e739.tar.gz |
COMPMID-2270: Implement POW operator for CL
Change-Id: Id14ecdc62439d90eb247bb75990d6593637cb42e
Signed-off-by: Usama Arif <usama.arif@arm.com>
Reviewed-on: https://review.mlplatform.org/c/1129
Comments-Addressed: Arm Jenkins <bsgcomp@arm.com>
Reviewed-by: Gian Marco Iodice <gianmarco.iodice@arm.com>
Tested-by: Arm Jenkins <bsgcomp@arm.com>
Diffstat (limited to 'src/core')
-rw-r--r-- | src/core/CL/CLKernelLibrary.cpp | 1 | ||||
-rw-r--r-- | src/core/CL/cl_kernels/elementwise_operation.cl | 3 | ||||
-rw-r--r-- | src/core/CL/kernels/CLElementwiseOperationKernel.cpp | 19 |
3 files changed, 13 insertions, 10 deletions
diff --git a/src/core/CL/CLKernelLibrary.cpp b/src/core/CL/CLKernelLibrary.cpp index b2d7e23624..e102f44b88 100644 --- a/src/core/CL/CLKernelLibrary.cpp +++ b/src/core/CL/CLKernelLibrary.cpp @@ -252,6 +252,7 @@ const std::map<std::string, std::string> CLKernelLibrary::_kernel_program_map = { "elementwise_operation_MIN", "elementwise_operation.cl" }, { "elementwise_operation_DIV", "elementwise_operation.cl" }, { "elementwise_operation_SQUARED_DIFF", "elementwise_operation.cl" }, + { "elementwise_operation_POWER", "elementwise_operation.cl" }, { "elementwise_operation_ADD_quantized", "elementwise_operation_quantized.cl" }, { "elementwise_operation_SUB_quantized", "elementwise_operation_quantized.cl" }, { "elementwise_operation_MAX_quantized", "elementwise_operation_quantized.cl" }, diff --git a/src/core/CL/cl_kernels/elementwise_operation.cl b/src/core/CL/cl_kernels/elementwise_operation.cl index 00d7ed3ba1..0b660e4012 100644 --- a/src/core/CL/cl_kernels/elementwise_operation.cl +++ b/src/core/CL/cl_kernels/elementwise_operation.cl @@ -1,5 +1,5 @@ /* - * Copyright (c) 2018 ARM Limited. + * Copyright (c) 2018-2019 ARM Limited. * * SPDX-License-Identifier: MIT * @@ -37,6 +37,7 @@ #define MIN(x, y) min(x, y) #define SQUARED_DIFF(x, y) (x - y) * (x - y) #define DIV(x, y) (x / y) +#define POWER(x, y) pow(x, y) #define OP_FUN_NAME_STR(op) elementwise_operation_##op #define OP_FUN_NAME(op) OP_FUN_NAME_STR(op) diff --git a/src/core/CL/kernels/CLElementwiseOperationKernel.cpp b/src/core/CL/kernels/CLElementwiseOperationKernel.cpp index 63c9244961..ce0c51dac5 100644 --- a/src/core/CL/kernels/CLElementwiseOperationKernel.cpp +++ b/src/core/CL/kernels/CLElementwiseOperationKernel.cpp @@ -42,6 +42,7 @@ std::map<ArithmeticOperation, std::string> supported_arithmetic_ops = { ArithmeticOperation::SQUARED_DIFF, "SQUARED_DIFF" }, { ArithmeticOperation::MIN, "MIN" }, { ArithmeticOperation::MAX, "MAX" }, + { ArithmeticOperation::POWER, "POWER" }, }; std::map<ArithmeticOperation, std::string> supported_sat_arithmetic_ops = @@ -64,7 +65,7 @@ std::string generate_id_for_tuning_common(const std::string &kernel_name, const return config_id; } -Status validate_arguments_with_division_rules(const ITensorInfo &input1, const ITensorInfo &input2, const ITensorInfo &output) +Status validate_arguments_with_float_only_supported_rules(const ITensorInfo &input1, const ITensorInfo &input2, const ITensorInfo &output) { ARM_COMPUTE_RETURN_ERROR_ON_NULLPTR(&input1, &input2, &output); ARM_COMPUTE_RETURN_ERROR_ON_F16_UNSUPPORTED(&input1); @@ -344,10 +345,10 @@ void CLArithmeticOperationKernel::configure(ArithmeticOperation op, const ICLTen Status CLArithmeticOperationKernel::validate(ArithmeticOperation op, const ITensorInfo *input1, const ITensorInfo *input2, const ITensorInfo *output) { ARM_COMPUTE_RETURN_ERROR_ON_NULLPTR(input1, input2, output); - if(op == ArithmeticOperation::DIV) + if(op == ArithmeticOperation::DIV || op == ArithmeticOperation::POWER) { - // Division doesn't support integer arithmetic - ARM_COMPUTE_RETURN_ON_ERROR(validate_arguments_with_division_rules(*input1, *input2, *output)); + // Division and Power operators don't support integer arithmetic + ARM_COMPUTE_RETURN_ON_ERROR(validate_arguments_with_float_only_supported_rules(*input1, *input2, *output)); ARM_COMPUTE_RETURN_ON_ERROR(validate_and_configure_window_for_division(*input1->clone(), *input2->clone(), *output->clone()).first); } else @@ -360,9 +361,9 @@ Status CLArithmeticOperationKernel::validate(ArithmeticOperation op, const ITens } std::pair<Status, Window> CLArithmeticOperationKernel::validate_and_configure_window(ITensorInfo &input1, ITensorInfo &input2, ITensorInfo &output) { - if(_op == ArithmeticOperation::DIV) + if(_op == ArithmeticOperation::DIV || _op == ArithmeticOperation::POWER) { - // Division doesn't support integer arithmetic + // Division and Power operators don't support integer arithmetic return validate_and_configure_window_for_division(input1, input2, output); } else @@ -372,10 +373,10 @@ std::pair<Status, Window> CLArithmeticOperationKernel::validate_and_configure_wi } Status CLArithmeticOperationKernel::validate_arguments(const ITensorInfo &input1, const ITensorInfo &input2, const ITensorInfo &output) { - if(_op == ArithmeticOperation::DIV) + if(_op == ArithmeticOperation::DIV || _op == ArithmeticOperation::POWER) { - // Division doesn't support integer arithmetic - return validate_arguments_with_division_rules(input1, input2, output); + // Division and Power operators don't support integer arithmetic + return validate_arguments_with_float_only_supported_rules(input1, input2, output); } else { |