comaniac commented on issue #4570: [relay] Relay annotation and partitioning 
for external compilers
URL: https://github.com/apache/incubator-tvm/pull/4570#issuecomment-569204482
 
 
   > Ideally whether to keep batch norm or not should be configurable. Actually 
for my use case, I want batch norm to be decomposed and its multiplication 
folded into previous conv, so that I don't have to worry about supporting batch 
norm on my end. But I expect others have a special op for handling fused Conv + 
BN + Relu.
   > 
   > UPDATE: I realized this is already achieved by allowing each backend to 
have its own annotate_compiler registration.
   
   Yes you are right. This PR provides an interface (annotation) for 
specialized codegen to do subgraph pattern matching for fusion or similar 
optimization. However, we also realize that this may be painful for developers 
and it might be better to provide a more convenient pattern matching for widely 
used cases (e.g., Conv+BN+ReLU as you mentioned). Our follow-up plans to focus 
more on the graph partitioning algorithm, and this is one of the cases we want 
to cover.

----------------------------------------------------------------
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