Removed FixRtInfo pass

ilya-lavrenov · Apr 13, 2023 · 1968632 · 1968632
1 parent a016e4e
commit 1968632
Show file tree

Hide file tree

Showing 8 changed files with 1 addition and 96 deletions.
diff --git a/src/common/conditional_compilation/docs/develop_cc_for_new_component.md b/src/common/conditional_compilation/docs/develop_cc_for_new_component.md
@@ -69,7 +69,6 @@ It checks whether the code region in this module is active or inactive by the ma
 
 There is an example of `conditional_compilation_gen.h`:
 ```
-#define ov_pass_FixRtInfo_run_on_function 1
 #define ov_pass_GraphRewrite_run_on_model 1
 #define ov_pass_InitNodeInfo_run_on_function 1
 #define ov_pass_ConstantFolding_run_on_model 1

diff --git a/src/common/transformations/include/transformations/fix_rt_info.hpp b/src/common/transformations/include/transformations/fix_rt_info.hpp
diff --git a/src/common/transformations/src/transformations/fix_rt_info.cpp b/src/common/transformations/src/transformations/fix_rt_info.cpp
diff --git a/src/common/transformations/src/transformations/init_node_info.cpp b/src/common/transformations/src/transformations/init_node_info.cpp
@@ -10,7 +10,6 @@
 #include <vector>
 
 #include "itt.hpp"
-#include "transformations/fix_rt_info.hpp"
 #include "transformations/rt_info/fused_names_attribute.hpp"
 #include "transformations/rt_info/primitives_priority_attribute.hpp"
 
@@ -27,6 +26,5 @@ bool ov::pass::InitNodeInfo::run_on_model(const std::shared_ptr<ngraph::Function
         auto& rtInfo = node->get_rt_info();
         rtInfo.emplace(FusedNames::get_type_info_static(), FusedNames{node->get_friendly_name()});
     }
-    FixRtInfo{}.run_on_model(f);
     return false;
 }
diff --git a/src/core/CMakeLists.txt b/src/core/CMakeLists.txt
@@ -115,7 +115,6 @@ file(GLOB_RECURSE smart_reshape_srcs ${CMAKE_CURRENT_SOURCE_DIR}/src/pass/smart_
 file(GLOB_RECURSE rt_info_srcs ${CMAKE_CURRENT_SOURCE_DIR}/src/pass/rt_info/*.cpp)
 set_source_files_properties("${CMAKE_CURRENT_SOURCE_DIR}/src/pass/convert_precision.cpp"
                             "${CMAKE_CURRENT_SOURCE_DIR}/src/pass/convert_fp32_to_fp16.cpp"
-                            "${CMAKE_CURRENT_SOURCE_DIR}/src/pass/fix_rt_info.cpp"
                             "${CMAKE_CURRENT_SOURCE_DIR}/src/pass/init_node_info.cpp"
                             "${CMAKE_CURRENT_SOURCE_DIR}/src/pass/serialize.cpp"
                             "${CMAKE_CURRENT_SOURCE_DIR}/src/op/type_relaxed.cpp"

diff --git a/src/inference/src/cnn_network_ngraph_impl.cpp b/src/inference/src/cnn_network_ngraph_impl.cpp
@@ -29,7 +29,6 @@
 #include "transformations/common_optimizations/nop_elimination.hpp"
 #include "transformations/common_optimizations/remove_concat_zero_dim_input.hpp"
 #include "transformations/common_optimizations/remove_multi_subgraph_op_dangling_params.hpp"
-#include "transformations/fix_rt_info.hpp"
 #include "transformations/smart_reshape/set_batch_size.hpp"
 #include "transformations/smart_reshape/smart_reshape.hpp"
 #include "transformations/utils/utils.hpp"
@@ -136,7 +135,6 @@ CNNNetworkNGraphImpl::CNNNetworkNGraphImpl(const std::shared_ptr<Function>& nGra
     {
         ov::pass::Manager m;
         using namespace ov::pass;
-        REGISTER_PASS(m, FixRtInfo)
         REGISTER_PASS(m, EliminateScatterUpdate)
         REGISTER_PASS(m, RemoveConcatZeroDimInput)
         REGISTER_PASS(m, RemoveMultiSubGraphOpDanglingParamsResults)
@@ -199,12 +197,6 @@ CNNNetworkNGraphImpl::CNNNetworkNGraphImpl(const CNNNetwork& network) {
     }
 
     _ngraph_function = ngraph::clone_function(*network.getFunction());
-    {
-        ov::pass::Manager m;
-        using namespace ov::pass;
-        REGISTER_PASS(m, FixRtInfo)
-        m.run_passes(_ngraph_function);
-    }
     validateFunctionNames();
     InputsDataMap inputs = network.getInputsInfo();
     OutputsDataMap outputs = network.getOutputsInfo();

diff --git a/src/inference/src/compilation_context.cpp b/src/inference/src/compilation_context.cpp
@@ -18,7 +18,6 @@
 #include "ie_itt.hpp"
 #include "ngraph/opsets/opset6.hpp"
 #include "openvino/pass/manager.hpp"
-#include "transformations/fix_rt_info.hpp"
 #include "transformations/hash.hpp"
 #include "transformations/rt_info/fused_names_attribute.hpp"
 #include "transformations/rt_info/primitives_priority_attribute.hpp"
@@ -86,7 +85,6 @@ std::string ModelCache::compute_hash(const std::shared_ptr<const ov::Model>& mod
     uint64_t seed = 0;
     // 1. Calculate hash on function
     ov::pass::Manager m;
-    m.register_pass<ov::pass::FixRtInfo>();
     m.register_pass<ov::pass::Hash>(seed);
     m.run_passes(std::const_pointer_cast<ov::Model>(model));
 

diff --git a/src/inference/tests/unit/ie_compilation_context_test.cpp b/src/inference/tests/unit/ie_compilation_context_test.cpp
@@ -189,8 +189,7 @@ TEST(NetworkContext, HashWithPrimitivesPriority) {
     op3["PrimitivesPriority"] = "testPriority";
 
     ASSERT_NE(ModelCache::compute_hash(net1, {}), ModelCache::compute_hash(net2, {}));
-
-    ASSERT_EQ(ModelCache::compute_hash(net2, {}), ModelCache::compute_hash(net3, {}));
+    ASSERT_NE(ModelCache::compute_hash(net2, {}), ModelCache::compute_hash(net3, {}));
 }
 
 TEST(NetworkContext, HashWithFusedNames) {