From fbf3ecc1833b860019f965c88cda87ec9f44c3d5 Mon Sep 17 00:00:00 2001 From: Gian Marco Iodice Date: Thu, 23 Aug 2018 17:26:21 +0100 Subject: COMPMID-1534 - Fix GEMM and Magnitude test for FP16 On GEMM we had accuracy issue On Magnitude we have disabled the fp16 acceleration since we do not have feature parity with CL and this function is not used for ML Change-Id: Iaebe3bbbd2a9f45db0c714aa5ebaf48eb0b65741 Reviewed-on: https://eu-gerrit-1.euhpc.arm.com/145467 Tested-by: Jenkins Reviewed-by: Georgios Pinitas Reviewed-by: Anthony Barbier --- src/runtime/NEON/functions/NEMagnitude.cpp | 36 ++++++++---------------------- 1 file changed, 9 insertions(+), 27 deletions(-) (limited to 'src/runtime/NEON/functions/NEMagnitude.cpp') diff --git a/src/runtime/NEON/functions/NEMagnitude.cpp b/src/runtime/NEON/functions/NEMagnitude.cpp index f86505449f..2738201d54 100644 --- a/src/runtime/NEON/functions/NEMagnitude.cpp +++ b/src/runtime/NEON/functions/NEMagnitude.cpp @@ -1,5 +1,5 @@ /* - * Copyright (c) 2016, 2017 ARM Limited. + * Copyright (c) 2016-2018 ARM Limited. * * SPDX-License-Identifier: MIT * @@ -31,36 +31,18 @@ using namespace arm_compute; -void NEMagnitude::configure(const ITensor *input1, const ITensor *input2, ITensor *output, MagnitudeType mag_type, bool use_fp16) +void NEMagnitude::configure(const ITensor *input1, const ITensor *input2, ITensor *output, MagnitudeType mag_type) { - if(use_fp16) + if(mag_type == MagnitudeType::L1NORM) { - if(mag_type == MagnitudeType::L1NORM) - { - auto k = arm_compute::support::cpp14::make_unique>(); - k->configure(input1, input2, output, nullptr); - _kernel = std::move(k); - } - else - { - auto k = arm_compute::support::cpp14::make_unique>(); - k->configure(input1, input2, output, nullptr); - _kernel = std::move(k); - } + auto k = arm_compute::support::cpp14::make_unique>(); + k->configure(input1, input2, output, nullptr); + _kernel = std::move(k); } else { - if(mag_type == MagnitudeType::L1NORM) - { - auto k = arm_compute::support::cpp14::make_unique>(); - k->configure(input1, input2, output, nullptr); - _kernel = std::move(k); - } - else - { - auto k = arm_compute::support::cpp14::make_unique>(); - k->configure(input1, input2, output, nullptr); - _kernel = std::move(k); - } + auto k = arm_compute::support::cpp14::make_unique>(); + k->configure(input1, input2, output, nullptr); + _kernel = std::move(k); } } -- cgit v1.2.1