[GitHub] [tvm] vinx13 commented on a diff in pull request #14166: [Unity][BYOC] Add batch matmul support to Relax CUTLASS BYOC

["vinx13 (via GitHub)" <gi...@apache.org>]

vinx13 commented on code in PR #14166:
URL: https://github.com/apache/tvm/pull/14166#discussion_r1122270289


##########
src/relax/transform/fuse_ops.cc:
##########
@@ -900,7 +900,7 @@ class PatternBasedPartitioner : ExprVisitor {
   using Group = GraphPartitioner::Group;
   using GroupMap = OperatorFusor::GroupMap;
   using ExprVisitor::VisitExpr_;
-  using FCheckMatch = backend::PatternRegistryEntryNode::FCheckMatch;
+  using FCheckMatch = runtime::TypedPackedFunc<bool(const Map<DFPattern, Expr>&, const Expr&)>;

Review Comment:
   we can remove https://github.com/apache/tvm/blob/03c5f998b37afe41257e2c37b1e95e0ef5357ce8/src/relax/transform/fuse_ops.cc#L43 if it's no longer needed



-- 
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-unsubscribe@tvm.apache.org

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