commit decd08b89565b18067d229c8c25b6f3a3333c653
author Jim Flynn <jim.flynn@arm.com> Sun Mar 13 22:35:46 2022 +0000
committer Jim Flynn <jim.flynn@arm.com> Mon Mar 14 00:34:17 2022 +0000
tree 56176931968b83646b7e69f8e5b84ff7a65a24aa
parent 77b284e6988b9a131d6abb3140ec6663c2ae84ae

IVGCVSW-6847 replace armnn:Optional with arm::pipe::Optional in profiling code

Change-Id: I048c538d4f8c21770aec2b2751c934d9fa15a4dc
Signed-off-by: Jim Flynn <jim.flynn@arm.com>

include/armnn/backends/profiling/IBackendProfiling.hpp

diff --git a/include/armnn/backends/profiling/IBackendProfiling.hpp b/include/armnn/backends/profiling/IBackendProfiling.hpp
index 564bd61..9affab2 100644
--- a/include/armnn/backends/profiling/IBackendProfiling.hpp
+++ b/include/armnn/backends/profiling/IBackendProfiling.hpp
@@ -4,11 +4,10 @@
 //
 #pragma once
 
-#include <armnn/Optional.hpp>
-
 #include <armnn/profiling/ISendTimelinePacket.hpp>
 
 #include <common/include/IProfilingGuidGenerator.hpp>
+#include <common/include/Optional.hpp>
 
 #include <memory>
 #include <vector>
@@ -56,25 +55,25 @@
 
     virtual uint16_t RegisterDevice(const std::string& deviceName,
                                     uint16_t cores = 0,
-                                    const armnn::Optional<std::string>& parentCategoryName
-                                        = armnn::EmptyOptional()) = 0;
+                                    const arm::pipe::Optional<std::string>& parentCategoryName
+                                        = arm::pipe::EmptyOptional()) = 0;
 
     virtual uint16_t RegisterCounterSet(const std::string& counterSetName,
                                         uint16_t count = 0,
-                                        const armnn::Optional<std::string>& parentCategoryName
-                                            = armnn::EmptyOptional()) = 0;
+                                        const arm::pipe::Optional<std::string>& parentCategoryName
+                                            = arm::pipe::EmptyOptional()) = 0;
 
     virtual uint16_t RegisterCounter(const uint16_t uid,
-                                     const std::string& parentCategoryName,
-                                     uint16_t counterClass,
-                                     uint16_t interpolation,
-                                     double multiplier,
-                                     const std::string& name,
-                                     const std::string& description,
-                                     const armnn::Optional<std::string>& units      = armnn::EmptyOptional(),
-                                     const armnn::Optional<uint16_t>& numberOfCores = armnn::EmptyOptional(),
-                                     const armnn::Optional<uint16_t>& deviceUid     = armnn::EmptyOptional(),
-                                     const armnn::Optional<uint16_t>& counterSetUid = armnn::EmptyOptional()) = 0;
+        const std::string& parentCategoryName,
+        uint16_t counterClass,
+        uint16_t interpolation,
+        double multiplier,
+        const std::string& name,
+        const std::string& description,
+        const arm::pipe::Optional<std::string>& units      = arm::pipe::EmptyOptional(),
+        const arm::pipe::Optional<uint16_t>& numberOfCores = arm::pipe::EmptyOptional(),
+        const arm::pipe::Optional<uint16_t>& deviceUid     = arm::pipe::EmptyOptional(),
+        const arm::pipe::Optional<uint16_t>& counterSetUid = arm::pipe::EmptyOptional()) = 0;
 
     virtual ~IRegisterBackendCounters() {}
 };

include/armnn/backends/profiling/IBackendProfilingContext.hpp

diff --git a/include/armnn/backends/profiling/IBackendProfilingContext.hpp b/include/armnn/backends/profiling/IBackendProfilingContext.hpp
index 0b0b0fa..a1ed05e 100644
--- a/include/armnn/backends/profiling/IBackendProfilingContext.hpp
+++ b/include/armnn/backends/profiling/IBackendProfilingContext.hpp
@@ -18,8 +18,8 @@
     virtual ~IBackendProfilingContext()
     {}
     virtual uint16_t RegisterCounters(uint16_t currentMaxGlobalCounterID) = 0;
-    virtual armnn::Optional<std::string> ActivateCounters(uint32_t capturePeriod, const std::vector<uint16_t>&
-        counterIds) = 0;
+    virtual arm::pipe::Optional<std::string> ActivateCounters(
+        uint32_t capturePeriod, const std::vector<uint16_t>& counterIds) = 0;
     virtual std::vector<Timestamp> ReportCounterValues() = 0;
     virtual bool EnableProfiling(bool flag) = 0;
     virtual bool EnableTimelineReporting(bool flag) = 0;

include/armnnTestUtils/MockBackend.hpp

diff --git a/include/armnnTestUtils/MockBackend.hpp b/include/armnnTestUtils/MockBackend.hpp
index 425062a..266c049 100644
--- a/include/armnnTestUtils/MockBackend.hpp
+++ b/include/armnnTestUtils/MockBackend.hpp
@@ -162,7 +162,7 @@
         return nextMaxGlobalCounterId;
     }
 
-    Optional<std::string> ActivateCounters(uint32_t capturePeriod, const std::vector<uint16_t>& counterIds)
+    arm::pipe::Optional<std::string> ActivateCounters(uint32_t capturePeriod, const std::vector<uint16_t>& counterIds)
     {
         if (capturePeriod == 0 || counterIds.size() == 0)
         {
@@ -170,11 +170,11 @@
         }
         else if (capturePeriod == 15939u)
         {
-            return armnn::Optional<std::string>("ActivateCounters example test error");
+            return arm::pipe::Optional<std::string>("ActivateCounters example test error");
         }
         m_CapturePeriod  = capturePeriod;
         m_ActiveCounters = counterIds;
-        return armnn::Optional<std::string>();
+        return arm::pipe::Optional<std::string>();
     }
 
     std::vector<arm::pipe::Timestamp> ReportCounterValues()