diff options
author | Sang-Hoon Park <sang-hoon.park@arm.com> | 2019-11-11 17:33:28 +0000 |
---|---|---|
committer | Sang-Hoon Park <sang-hoon.park@arm.com> | 2019-11-12 11:23:42 +0000 |
commit | eaa01ab593428bc7267ebbe107b2d813a11b64b5 (patch) | |
tree | 80e0af6b7ed0a82387080d6abb6c0a04d9f1d509 /src/runtime/CL | |
parent | 75041a1cb81c59a5a5ddd9b708476c0142362d9e (diff) | |
download | ComputeLibrary-eaa01ab593428bc7267ebbe107b2d813a11b64b5.tar.gz |
COMPMID-2671 use Signed32 for default output data type of ArgMinMax
Signed32 is used as data types before and after reshaping of ArgMinMax.
Change-Id: I230af43a931d4e106de6c72f716ced1dab511084
Signed-off-by: Sang-Hoon Park <sang-hoon.park@arm.com>
Reviewed-on: https://review.mlplatform.org/c/2262
Comments-Addressed: Arm Jenkins <bsgcomp@arm.com>
Reviewed-by: Michele Di Giorgio <michele.digiorgio@arm.com>
Tested-by: Arm Jenkins <bsgcomp@arm.com>
Reviewed-by: Giorgio Arena <giorgio.arena@arm.com>
Diffstat (limited to 'src/runtime/CL')
-rw-r--r-- | src/runtime/CL/functions/CLReductionOperation.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/runtime/CL/functions/CLReductionOperation.cpp b/src/runtime/CL/functions/CLReductionOperation.cpp index 447c15b1e8..3aa5a813b6 100644 --- a/src/runtime/CL/functions/CLReductionOperation.cpp +++ b/src/runtime/CL/functions/CLReductionOperation.cpp @@ -86,7 +86,7 @@ Status CLReductionOperation::validate(const ITensorInfo *input, const ITensorInf const auto input_data_type = input->data_type(); const auto input_num_channles = input->num_channels(); const auto input_qinfo = input->quantization_info(); - const auto output_data_type = is_arg_min_max ? DataType::U32 : output->data_type(); + const auto output_data_type = is_arg_min_max ? DataType::S32 : output->data_type(); auto initialize_tensorinfo = [](TensorInfo & ti, TensorShape shape, DataType data_type, int num_channels, QuantizationInfo qinfo) { @@ -208,7 +208,7 @@ ICLTensor *CLReductionOperation::configure_intermediate_result_vector(ICLTensor if(is_arg_min_max) { - _results_vector.back().info()->set_data_type(DataType::U32).set_is_resizable(true).reset_padding(); + _results_vector.back().info()->set_data_type(DataType::S32).set_is_resizable(true).reset_padding(); } return _is_reshape_required ? &_results_vector.back() : output; @@ -229,7 +229,7 @@ void CLReductionOperation::configure(ICLTensor *input, ICLTensor *output, unsign if(_is_reshape_required) { const TensorShape output_shape = arm_compute::misc::shape_calculator::compute_reduced_shape(input->info()->tensor_shape(), axis, false); - const auto output_data_type = is_arg_min_max ? DataType::U32 : input->info()->data_type(); + const auto output_data_type = is_arg_min_max ? DataType::S32 : input->info()->data_type(); auto_init_if_empty(*output->info(), input->info()->clone()->set_tensor_shape(output_shape).set_data_type(output_data_type).reset_padding().set_is_resizable(true)); } |