diff options
author | Michalis Spyrou <michalis.spyrou@arm.com> | 2019-05-01 13:03:59 +0100 |
---|---|---|
committer | Michalis Spyrou <michalis.spyrou@arm.com> | 2019-05-10 13:30:46 +0000 |
commit | bcfd09a14a61bb8457555c61e7c5697b1f901ddb (patch) | |
tree | 30754b3eaad2464c787d74b304bcd7b135dd4178 /tests/validation/reference/ArithmeticOperations.cpp | |
parent | 46a49a0a8206f0efa7afd514940e180a88ffd732 (diff) | |
download | ComputeLibrary-bcfd09a14a61bb8457555c61e7c5697b1f901ddb.tar.gz |
COMPMID-2177 Fix clang warnings
Change-Id: I78039db8c58d7b14a042c41e54c25fb9cb509bf7
Signed-off-by: Michalis Spyrou <michalis.spyrou@arm.com>
Reviewed-on: https://review.mlplatform.org/c/1092
Reviewed-by: VidhyaSudhan Loganathan <vidhyasudhan.loganathan@arm.com>
Comments-Addressed: Arm Jenkins <bsgcomp@arm.com>
Tested-by: Arm Jenkins <bsgcomp@arm.com>
Diffstat (limited to 'tests/validation/reference/ArithmeticOperations.cpp')
-rw-r--r-- | tests/validation/reference/ArithmeticOperations.cpp | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/tests/validation/reference/ArithmeticOperations.cpp b/tests/validation/reference/ArithmeticOperations.cpp index 062be93b95..a6205af2c6 100644 --- a/tests/validation/reference/ArithmeticOperations.cpp +++ b/tests/validation/reference/ArithmeticOperations.cpp @@ -1,5 +1,5 @@ /* - * Copyright (c) 2017-2018 ARM Limited. + * Copyright (c) 2017-2019 ARM Limited. * * SPDX-License-Identifier: MIT * @@ -88,7 +88,9 @@ struct BroadcastUnroll<0> template <typename T> SimpleTensor<T> arithmetic_operation(ArithmeticOperation op, const SimpleTensor<T> &src1, const SimpleTensor<T> &src2, SimpleTensor<T> &dst, ConvertPolicy convert_policy) { - Coordinates id_src1, id_src2, id_dst; + Coordinates id_src1{}; + Coordinates id_src2{}; + Coordinates id_dst{}; BroadcastUnroll<Coordinates::num_max_dimensions>::unroll(op, src1, src2, dst, convert_policy, id_src1, id_src2, id_dst); @@ -98,14 +100,16 @@ SimpleTensor<T> arithmetic_operation(ArithmeticOperation op, const SimpleTensor< template <> SimpleTensor<uint8_t> arithmetic_operation(ArithmeticOperation op, const SimpleTensor<uint8_t> &src1, const SimpleTensor<uint8_t> &src2, SimpleTensor<uint8_t> &dst, ConvertPolicy convert_policy) { + Coordinates id_src1{}; + Coordinates id_src2{}; + Coordinates id_dst{}; + if(dst.data_type() == DataType::QASYMM8) { SimpleTensor<float> src1_tmp = convert_from_asymmetric(src1); SimpleTensor<float> src2_tmp = convert_from_asymmetric(src2); SimpleTensor<float> dst_tmp(TensorShape::broadcast_shape(src1.shape(), src2.shape()), dst.data_type()); - Coordinates id_src1, id_src2, id_dst; - BroadcastUnroll<Coordinates::num_max_dimensions>::unroll(op, src1_tmp, src2_tmp, dst_tmp, convert_policy, id_src1, id_src2, id_dst); dst = convert_to_asymmetric(dst_tmp, dst.quantization_info()); @@ -114,8 +118,6 @@ SimpleTensor<uint8_t> arithmetic_operation(ArithmeticOperation op, const SimpleT else { // DataType::U8 - Coordinates id_src1, id_src2, id_dst; - BroadcastUnroll<Coordinates::num_max_dimensions>::unroll(op, src1, src2, dst, convert_policy, id_src1, id_src2, id_dst); return dst; |