Commit 06916cbc authored by Nick Korovaiko's avatar Nick Korovaiko Committed by Robert Kimball

Abort messages in Matcher to better understand cases where we fail to match (#2179)

*  abort messages in matcher.cpp

* style fixes
parent 1feb49f1
...@@ -69,6 +69,12 @@ namespace ngraph ...@@ -69,6 +69,12 @@ namespace ngraph
pattern_map[label] = graph_node; pattern_map[label] = graph_node;
} }
} }
if (!is_match)
{
NGRAPH_DEBUG << "[MATCHER] Aborting at " << graph_node->get_name()
<< " for pattern " << label->get_name();
}
return is_match; return is_match;
} }
...@@ -131,6 +137,8 @@ namespace ngraph ...@@ -131,6 +137,8 @@ namespace ngraph
} }
else else
{ {
NGRAPH_DEBUG << "[MATCHER] Aborting at " << graph_node->get_name()
<< " for pattern " << any->get_name();
return false; return false;
} }
} }
...@@ -157,10 +165,14 @@ namespace ngraph ...@@ -157,10 +165,14 @@ namespace ngraph
} }
} }
NGRAPH_DEBUG << "[MATCHER] Aborting at " << graph_node->get_name()
<< " for pattern " << any->get_name();
return false; return false;
} }
else else
{ {
NGRAPH_DEBUG << "[MATCHER] Aborting at " << graph_node->get_name()
<< " for pattern " << any->get_name();
return false; return false;
} }
} }
...@@ -211,6 +223,8 @@ namespace ngraph ...@@ -211,6 +223,8 @@ namespace ngraph
match_arguments(pattern_node, graph_node, pattern_map)); match_arguments(pattern_node, graph_node, pattern_map));
} }
NGRAPH_DEBUG << "[MATCHER] Aborting at " << graph_node->get_name() << " for pattern "
<< pattern_node->get_name();
return abort_match(watermark, false); return abort_match(watermark, false);
} }
...@@ -244,6 +258,8 @@ namespace ngraph ...@@ -244,6 +258,8 @@ namespace ngraph
if (args.size() != pattern_args.size()) if (args.size() != pattern_args.size())
{ {
NGRAPH_DEBUG << "[MATCHER] Aborting at " << graph_node->get_name()
<< " for pattern " << pattern_node->get_name();
return false; return false;
} }
...@@ -273,6 +289,9 @@ namespace ngraph ...@@ -273,6 +289,9 @@ namespace ngraph
return true; return true;
} }
} }
NGRAPH_DEBUG << "[MATCHER] Aborting at " << graph_node->get_name() << " for pattern "
<< pattern_node->get_name();
return false; return false;
} }
...@@ -393,6 +412,8 @@ namespace ngraph ...@@ -393,6 +412,8 @@ namespace ngraph
if (!matched) if (!matched)
{ {
NGRAPH_DEBUG << "[RecurrentMatcher] Aborting at " << graph->get_name()
<< " for pattern " << m_pattern->get_name();
m_match_root.reset(); m_match_root.reset();
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment