[GitHub] [tvm] masahi commented on a diff in pull request #14439: [Unity][Graph matching] Automatically add `used-by` constraints for `is_op` pattern

[via GitHub]

masahi commented on code in PR #14439:
URL: https://github.com/apache/tvm/pull/14439#discussion_r1153724926


##########
include/tvm/relax/dataflow_matcher.h:
##########
@@ -65,15 +65,6 @@ TVM_DLL tvm::runtime::Map<DFPattern, Var> MatchGraph(const 
PatternContext& ctx,
                                                      Optional<Var> start_hint 
= NullOpt,
                                                      bool must_include_hint = 
false);
 
-/**
- * \brief Match a graph-wise pattern with the current context 
(PatternContext::Current()).
- */
-inline tvm::runtime::Map<DFPattern, Var> MatchGraphDefault(const 
DataflowBlock& dfb,
-                                                           Optional<Var> 
start_hint = NullOpt,
-                                                           bool 
must_include_hint = false) {
-  return MatchGraph(PatternContext::Current(), dfb, start_hint, 
must_include_hint);

Review Comment:
   This gets removed since it is not used anywhere but it touches 
`PatternContext::Current()` (which is now `Optional`)



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@tvm.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org