Remove GetGraph and include of Graph.hpp header from public header

Remove deprecated GetGraph() from OptimizationViews. This method has
been deprecated for a long time and no backends still need it. Remove
include of Graph.hpp from the public headers.

Add includes elsewhere to deal with the header fallout.

Signed-off-by: Matthew Bentham <matthew.bentham@arm.com>
Change-Id: I8dae275a8a446d9d0e19be62684e9b3cd2fa493d

include/armnn/backends/OptimizationViews.hpp

@@ -10,7 +10,6 @@
 #include <armnn/INetwork.hpp>
 #include <armnn/backends/SubgraphView.hpp>
 #include <vector>
-#include "Graph.hpp"
 
 namespace armnn
 {
@@ -61,9 +60,6 @@ class OptimizationViews
 
     bool Validate(const SubgraphView& originalSubgraph) const;
 
-    ARMNN_DEPRECATED_MSG_REMOVAL_DATE("GetGraph is deprecated. Use GetINetwork instead.", "23.08")
-    Graph& GetGraph() { return m_Graph; }
-
     INetwork* GetINetwork() { return m_INetwork.get(); }
     INetwork& GetINetworkRef() { return *m_INetwork; }
 
@@ -72,9 +68,6 @@ class OptimizationViews
     Subgraphs m_FailedOptimizations;            ///< Subgraphs from the original subgraph which cannot be supported
     Subgraphs m_UntouchedSubgraphs;             ///< Subgraphs from the original subgraph which remain unmodified
 
-    /// Graph object used only as a container for any layer generated by the optimization process
-    Graph m_Graph;
-
     /// INetworkPtr object used only as a container for any layer generated by the optimization process
     /// Also, can use to AddPrecompiledLayer to the SubstitutionPair
     /// Use in favour of m_Graph which depreciates in 23.08

src/armnnTestUtils/MockBackend.cpp

@@ -11,6 +11,8 @@
 #include <backendsCommon/test/MockBackendId.hpp>
 #include <SubgraphViewSelector.hpp>
 
+#include "Layer.hpp"
+
 namespace armnn
 {
 
@@ -301,4 +303,4 @@ std::unique_ptr<ICustomAllocator> MockBackend::GetDefaultAllocator() const
     return std::make_unique<DefaultAllocator>();
 }
 
-}    // namespace armnn
+}    // namespace armnn

src/backends/backendsCommon/IBackendInternal.cpp

@@ -5,6 +5,7 @@
 
 #include <armnn/BackendOptions.hpp>
 #include <armnn/backends/IBackendInternal.hpp>
+#include <armnn/backends/WorkloadFactory.hpp>
 
 namespace armnn
 {

src/backends/backendsCommon/test/TestDynamicBackend.cpp

@@ -6,7 +6,7 @@
 #include "TestDynamicBackend.hpp"
 
 #include <armnn/backends/IBackendInternal.hpp>
-
+#include <armnn/backends/WorkloadFactory.hpp>
 #include <armnn/utility/IgnoreUnused.hpp>
 
 constexpr const char* TestDynamicBackendId()

src/backends/backendsCommon/test/mockBackend/MockImportBackend.hpp

@@ -5,6 +5,7 @@
 #pragma once
 
 #include <armnn/backends/IBackendInternal.hpp>
+#include <armnn/utility/PolymorphicDowncast.hpp>
 
 namespace armnn
 {

src/backends/cl/test/ClBackendTests.cpp

@@ -3,6 +3,7 @@
 // SPDX-License-Identifier: MIT
 //
 
+#include <backendsCommon/TensorHandleFactoryRegistry.hpp>
 #include <cl/ClBackend.hpp>
 #include <cl/ClTensorHandleFactory.hpp>
 #include <cl/ClImportTensorHandleFactory.hpp>

src/backends/neon/test/NeonBackendTests.cpp

@@ -3,6 +3,7 @@
 // SPDX-License-Identifier: MIT
 //
 
+#include <backendsCommon/TensorHandleFactoryRegistry.hpp>
 #include <neon/NeonBackend.hpp>
 #include <neon/NeonTensorHandleFactory.hpp>
 

src/backends/reference/test/RefBackendTests.cpp

@@ -3,9 +3,11 @@
 // SPDX-License-Identifier: MIT
 //
 
+#include <backendsCommon/TensorHandleFactoryRegistry.hpp>
 #include <reference/RefBackend.hpp>
 #include <reference/RefTensorHandleFactory.hpp>
 
+#include <armnn/backends/WorkloadFactory.hpp>
 #include <doctest/doctest.h>
 
 using namespace armnn;