From: janeil01 Date: Thu, 7 Nov 2019 09:32:28 +0000 (+0000) Subject: IVGCVSW-4102 Move ProfilingGuid to public interface X-Git-Tag: submit/tizen/20200316.035456~69 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=c4946c7addd65f032951dbf16fb824cdd67fd463;p=platform%2Fupstream%2Farmnn.git IVGCVSW-4102 Move ProfilingGuid to public interface * Moved ProfilingGuid to Types.hpp * Refactoring to enable ProfilingGuid Signed-off-by: janeil01 Change-Id: Ibf77002d74e484f8a63ffd96aa14303c1f0d38ae --- diff --git a/CMakeLists.txt b/CMakeLists.txt index 5b80430..6856da6 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -485,7 +485,6 @@ list(APPEND armnn_sources src/profiling/ProfilingConnectionDumpToFileDecorator.hpp src/profiling/ProfilingConnectionFactory.cpp src/profiling/ProfilingConnectionFactory.hpp - src/profiling/ProfilingGuid.hpp src/profiling/ProfilingGuidGenerator.hpp src/profiling/ProfilingService.cpp src/profiling/ProfilingService.hpp diff --git a/include/armnn/Types.hpp b/include/armnn/Types.hpp index 51162e6..b2aa52e 100644 --- a/include/armnn/Types.hpp +++ b/include/armnn/Types.hpp @@ -7,6 +7,7 @@ #include #include #include +#include #include "BackendId.hpp" #include "Exceptions.hpp" @@ -219,4 +220,96 @@ class ITensorHandle; /// @param tensorHandle - TensorHandle for the input tensor to the Debug layer using DebugCallbackFunction = std::function; + +namespace profiling +{ + +class ProfilingGuid +{ +public: + ProfilingGuid(uint64_t guid) : m_Guid(guid) {} + + operator uint64_t() const { return m_Guid; } + + bool operator==(const ProfilingGuid& other) const + { + return m_Guid == other.m_Guid; + } + + bool operator!=(const ProfilingGuid& other) const + { + return m_Guid != other.m_Guid; + } + + bool operator<(const ProfilingGuid& other) const + { + return m_Guid < other.m_Guid; + } + + bool operator<=(const ProfilingGuid& other) const + { + return m_Guid <= other.m_Guid; + } + + bool operator>(const ProfilingGuid& other) const + { + return m_Guid > other.m_Guid; + } + + bool operator>=(const ProfilingGuid& other) const + { + return m_Guid >= other.m_Guid; + } + +protected: + uint64_t m_Guid; +}; + +/// Strongly typed guids to distinguish between those generated at runtime, and those that are statically defined. +struct ProfilingDynamicGuid : public ProfilingGuid +{ + using ProfilingGuid::ProfilingGuid; +}; + +struct ProfilingStaticGuid : public ProfilingGuid +{ + using ProfilingGuid::ProfilingGuid; +}; + +} // namespace profiling + } // namespace armnn + + +namespace std +{ +// make ProfilingGuid hashable +template<> +struct hash +{ + std::size_t operator()(armnn::profiling::ProfilingGuid const& guid) const noexcept + { + return hash()(uint64_t(guid)); + } +}; + +// make ProfilingDynamicGuid hashable +template<> +struct hash +{ + std::size_t operator()(armnn::profiling::ProfilingDynamicGuid const& guid) const noexcept + { + return hash()(uint64_t(guid)); + } +}; + +// make ProfilingStaticGuid hashable +template<> +struct hash +{ + std::size_t operator()(armnn::profiling::ProfilingStaticGuid const& guid) const noexcept + { + return hash()(uint64_t(guid)); + } +}; +} diff --git a/src/profiling/IProfilingGuidGenerator.hpp b/src/profiling/IProfilingGuidGenerator.hpp index 34ee967..fb9f740 100644 --- a/src/profiling/IProfilingGuidGenerator.hpp +++ b/src/profiling/IProfilingGuidGenerator.hpp @@ -5,7 +5,7 @@ #pragma once -#include "ProfilingGuid.hpp" +#include #include diff --git a/src/profiling/LabelsAndEventClasses.hpp b/src/profiling/LabelsAndEventClasses.hpp index 07aeb81..daa33ad 100644 --- a/src/profiling/LabelsAndEventClasses.hpp +++ b/src/profiling/LabelsAndEventClasses.hpp @@ -5,9 +5,10 @@ #pragma once -#include "ProfilingGuid.hpp" #include "ProfilingGuidGenerator.hpp" +#include + namespace armnn { diff --git a/src/profiling/ProfilingGuid.hpp b/src/profiling/ProfilingGuid.hpp deleted file mode 100644 index 5deee11..0000000 --- a/src/profiling/ProfilingGuid.hpp +++ /dev/null @@ -1,70 +0,0 @@ -// -// Copyright © 2019 Arm Ltd. All rights reserved. -// SPDX-License-Identifier: MIT -// - -#pragma once - -#include - -namespace armnn -{ - -namespace profiling -{ - -class ProfilingGuid -{ -public: - ProfilingGuid(uint64_t guid) : m_Guid(guid) {} - - operator uint64_t() const { return m_Guid; } - - bool operator==(const ProfilingGuid& other) const - { - return m_Guid == other.m_Guid; - } - - bool operator!=(const ProfilingGuid& other) const - { - return m_Guid != other.m_Guid; - } - - bool operator<(const ProfilingGuid& other) const - { - return m_Guid < other.m_Guid; - } - - bool operator<=(const ProfilingGuid& other) const - { - return m_Guid <= other.m_Guid; - } - - bool operator>(const ProfilingGuid& other) const - { - return m_Guid > other.m_Guid; - } - - bool operator>=(const ProfilingGuid& other) const - { - return m_Guid >= other.m_Guid; - } - -protected: - uint64_t m_Guid; -}; - -/// Strongly typed guids to distinguish between those generated at runtime, and those that are statically defined. -struct ProfilingDynamicGuid : public ProfilingGuid -{ - using ProfilingGuid::ProfilingGuid; -}; - -struct ProfilingStaticGuid : public ProfilingGuid -{ - using ProfilingGuid::ProfilingGuid; -}; - -} // namespace profiling - -} // namespace armnn diff --git a/src/profiling/TimelineUtilityMethods.hpp b/src/profiling/TimelineUtilityMethods.hpp index 5bb4342..51fceb9 100644 --- a/src/profiling/TimelineUtilityMethods.hpp +++ b/src/profiling/TimelineUtilityMethods.hpp @@ -6,7 +6,7 @@ #pragma once #include "ISendTimelinePacket.hpp" -#include "ProfilingGuid.hpp" +#include namespace armnn { diff --git a/src/profiling/test/ProfilingGuidTest.cpp b/src/profiling/test/ProfilingGuidTest.cpp index fa01435..f72cc1c 100644 --- a/src/profiling/test/ProfilingGuidTest.cpp +++ b/src/profiling/test/ProfilingGuidTest.cpp @@ -3,7 +3,7 @@ // SPDX-License-Identifier: MIT // -#include "ProfilingGuid.hpp" +#include #include