From 910e41ff2099b0516fc9bff2baaa2ec678c8a7bf Mon Sep 17 00:00:00 2001 From: Ivan Tikhonov Date: Thu, 10 Sep 2020 13:36:11 +0300 Subject: [PATCH] move priorbox to ie transformations to Opset1ToLegacyOpset pipeline --- .../convert_prior_to_ie_prior.hpp | 4 ++-- .../common_optimizations/common_optimizations.cpp | 3 --- .../convert_opset1_to_legacy/convert_opset1_to_legacy.cpp | 2 ++ .../convert_prior_to_ie_prior.cpp | 8 ++++---- 4 files changed, 8 insertions(+), 9 deletions(-) diff --git a/inference-engine/src/transformations/include/transformations/convert_opset1_to_legacy/convert_prior_to_ie_prior.hpp b/inference-engine/src/transformations/include/transformations/convert_opset1_to_legacy/convert_prior_to_ie_prior.hpp index 6d1c7fbbb1f..6f0ba5f53bc 100644 --- a/inference-engine/src/transformations/include/transformations/convert_opset1_to_legacy/convert_prior_to_ie_prior.hpp +++ b/inference-engine/src/transformations/include/transformations/convert_opset1_to_legacy/convert_prior_to_ie_prior.hpp @@ -19,9 +19,9 @@ class TRANSFORMATIONS_API ConvertPriorBox; } // namespace pass } // namespace ngraph -class ngraph::pass::ConvertPriorBox: public ngraph::pass::GraphRewrite { +class ngraph::pass::ConvertPriorBox : public ngraph::pass::MatcherPass { public: - ConvertPriorBox() : GraphRewrite() { + ConvertPriorBox() : { convert_prior_box(); convert_prior_box_clustered(); } diff --git a/inference-engine/src/transformations/src/transformations/common_optimizations/common_optimizations.cpp b/inference-engine/src/transformations/src/transformations/common_optimizations/common_optimizations.cpp index f2383284de1..cad9d8e2708 100644 --- a/inference-engine/src/transformations/src/transformations/common_optimizations/common_optimizations.cpp +++ b/inference-engine/src/transformations/src/transformations/common_optimizations/common_optimizations.cpp @@ -7,7 +7,6 @@ #include "transformations/common_optimizations/algebraic_simplification.hpp" #include "transformations/common_optimizations/nop_elimination.hpp" #include "transformations/common_optimizations/common_optimizations.hpp" -#include "transformations/convert_opset1_to_legacy/convert_prior_to_ie_prior.hpp" #include "transformations/depth_to_space_fusion.hpp" #include "transformations/optimize_strided_slice.hpp" #include "transformations/convert_scatter_elements_to_scatter.hpp" @@ -33,8 +32,6 @@ bool ngraph::pass::CommonOptimizations::run_on_function(std::shared_ptr(); - manager.register_pass(); // WA: ConvertPriorBox must be executed before CF - manager.register_pass(); manager.register_pass(); // Resolves dynamism (replaces NonZero), CF needed manager.register_pass(); manager.register_pass(); diff --git a/inference-engine/src/transformations/src/transformations/convert_opset1_to_legacy/convert_opset1_to_legacy.cpp b/inference-engine/src/transformations/src/transformations/convert_opset1_to_legacy/convert_opset1_to_legacy.cpp index 25847676511..64ac5c7405b 100644 --- a/inference-engine/src/transformations/src/transformations/convert_opset1_to_legacy/convert_opset1_to_legacy.cpp +++ b/inference-engine/src/transformations/src/transformations/convert_opset1_to_legacy/convert_opset1_to_legacy.cpp @@ -24,6 +24,7 @@ #include #include #include +#include #include #include #include @@ -161,6 +162,7 @@ bool ngraph::pass::ConvertOpSet1ToLegacy::run_on_function(std::shared_ptradd_matcher(); anchor->add_matcher(); anchor->add_matcher(); + anchor->add_matcher(); anchor->set_name("ngraph::pass::ConvertOpSet1ToLegacy"); // List of final conversion transformations that must to be executed diff --git a/inference-engine/src/transformations/src/transformations/convert_opset1_to_legacy/convert_prior_to_ie_prior.cpp b/inference-engine/src/transformations/src/transformations/convert_opset1_to_legacy/convert_prior_to_ie_prior.cpp index b881df50b36..9ac93e4ecc7 100644 --- a/inference-engine/src/transformations/src/transformations/convert_opset1_to_legacy/convert_prior_to_ie_prior.cpp +++ b/inference-engine/src/transformations/src/transformations/convert_opset1_to_legacy/convert_prior_to_ie_prior.cpp @@ -33,7 +33,7 @@ void ngraph::pass::ConvertPriorBox::convert_prior_box() { auto prior_box = std::make_shared(data, image, attr); auto unsqueeze = std::make_shared (prior_box, axes); - ngraph::graph_rewrite_callback callback = [](pattern::Matcher& m) { + ngraph::matcher_pass_callback callback = [](pattern::Matcher& m) { auto unsqueeze = std::dynamic_pointer_cast (m.get_match_root()); if (!unsqueeze) { return false; @@ -152,7 +152,7 @@ void ngraph::pass::ConvertPriorBox::convert_prior_box() { }; auto m = std::make_shared(unsqueeze, "CPUFusion.ConvertPriorBoxToPriorBoxIE"); - this->add_matcher(m, callback, PassProperty::CHANGE_DYNAMIC_STATE); + this->register_matcher(m, callback, PassProperty::CHANGE_DYNAMIC_STATE); } void ngraph::pass::ConvertPriorBox::convert_prior_box_clustered() { @@ -172,7 +172,7 @@ void ngraph::pass::ConvertPriorBox::convert_prior_box_clustered() { auto prior_box = std::make_shared(data, image, attr); auto unsqueeze = std::make_shared (prior_box, axes); - ngraph::graph_rewrite_callback callback = [](pattern::Matcher& m) { + ngraph::matcher_pass_callback callback = [](pattern::Matcher& m) { auto unsqueeze = std::dynamic_pointer_cast (m.get_match_root()); if (!unsqueeze) { return false; @@ -290,5 +290,5 @@ void ngraph::pass::ConvertPriorBox::convert_prior_box_clustered() { }; auto m = std::make_shared(unsqueeze, "CPUFusion.ConvertPriorBoxClusteredToPriorBoxClusteredIE"); - this->add_matcher(m, callback, PassProperty::CHANGE_DYNAMIC_STATE); + this->register_matcher(m, callback, PassProperty::CHANGE_DYNAMIC_STATE); } \ No newline at end of file