huajsj commented on PR #11405: URL: https://github.com/apache/tvm/pull/11405#issuecomment-1136363802
> Hi, @huajsj. Thanks for your contribution on TVM. I have a simple question for the graph_split func. In many real use case, the relay function can be existed in the module['main'] (i.e. OP fusion or BYOC cases). How can we handle these cases to split the graph? As line 116 shown, graph_split only parse the top level function and treat the internal "relay.function" as a "relay.expr.call", that means the operator inside the function will not get involved by the graph split, and not impact the result. then either fuse or byoc logic will not get broken. And if there is a requirement to split the internal function of like BYOC, then the order should be that do the graph split first then do the BYOC annotation. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
