commit 67cc5fc08da79933361e7cd0af3b6452d2424a61
author David Monahan <david.monahan@arm.com> Wed Nov 03 12:56:41 2021 +0000
committer Colm Donelan <colm.donelan@arm.com> Wed Nov 03 14:06:55 2021 +0000
tree fba3d2f07ef09c3a8d0ddfd0cf9112d72e648f01
parent 2ea38c7f56d57a4fdcf709c9d61b7bdfab4ebfd9

Revert "IVGCVSW-6359 Added support for Float16 (Half) to Execute Network"

This reverts commit 2d9956162dd002a41f7fb4fa6753195d33524c7f.

Reason for revert: After some discussion, this does technically implement Float16 support for ExecuteNetwork, but not in a way which matches most use cases and is likely to cause issues in the future. Reverting for now.

Change-Id: I4ce6de6879216e694631f5dc68e46fb793fae0a9

tests/ExecuteNetwork/ExecuteNetwork.cpp

diff --git a/tests/ExecuteNetwork/ExecuteNetwork.cpp b/tests/ExecuteNetwork/ExecuteNetwork.cpp
index 0d52711..a0a08d3 100644
--- a/tests/ExecuteNetwork/ExecuteNetwork.cpp
+++ b/tests/ExecuteNetwork/ExecuteNetwork.cpp
@@ -12,7 +12,6 @@
 #include <armnnUtils/Filesystem.hpp>
 #include <armnnUtils/TContainer.hpp>
 #include <InferenceTest.hpp>
-#include <Half.hpp>
 
 #if defined(ARMNN_SERIALIZER)
 #include "armnnDeserializer/IDeserializer.hpp"
@@ -485,7 +484,7 @@
             armnn::DataType type = model.GetOutputBindingInfo(outputIdx).second.GetDataType();
             switch (type)
             {
-                // --output-type only supports float, float16, int,  qasymms8 or qasymmu8.
+                // --output-type only supports float, int,  qasymms8 or qasymmu8.
                 case armnn::DataType::Float32:
                     if (params.m_OutputTypes[outputIdx].compare("float") != 0)
                     {
@@ -494,14 +493,6 @@
                                            ". This may cause unexpected problems or random failures.";
                     }
                     break;
-                case armnn::DataType::Float16:
-                    if (params.m_OutputTypes[outputIdx].compare("float16") != 0)
-                    {
-                        ARMNN_LOG(warning) << "Model output index: " << outputIdx << " has data type Float16. The " <<
-                                           "corresponding --output-type is " << params.m_OutputTypes[outputIdx] <<
-                                           ". This may cause unexpected problems or random failures.";
-                    }
-                    break;
                 case armnn::DataType::QAsymmU8:
                     if (params.m_OutputTypes[outputIdx].compare("qasymmu8") != 0)
                     {
@@ -539,10 +530,6 @@
                 {
                     outputDataContainers.push_back(std::vector<float>(model.GetOutputSize(i)));
                 }
-                else if (params.m_OutputTypes[i].compare("float16") == 0)
-                {
-                    outputDataContainers.push_back(std::vector<armnn::Half>(model.GetOutputSize(i)));
-                }
                 else if (params.m_OutputTypes[i].compare("int") == 0)
                 {
                     outputDataContainers.push_back(std::vector<int>(model.GetOutputSize(i)));