mbarrett97 commented on issue #4771: [Relay] Added Merge Composite pass
URL: https://github.com/apache/incubator-tvm/pull/4771#issuecomment-578979321
 
 
   It's the intention that this can be called on the entire Relay graph so that 
it can be used to help implement a generic annotation pass (one that is aware 
of composite functions). That way we can define functions similar to the 'Is 
Supported?' mechanism in the original annotation PR (since taken down) where 
you could declare in Python whether an operator was supported. That could be 
extended to say whether a composite function is supported without having to add 
pattern matching code to the annotator.
   
   The problem there is the case where composite functions do not end up in an 
external function after partitioning. My thinking is to have some legalize pass 
after the partitioning that removes the composite functions from sections of 
the graph not marked as external.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to