mbs-octoml edited a comment on pull request #62:
URL: https://github.com/apache/tvm-rfcs/pull/62#issuecomment-1069524854


   Hey all, heads up I'm taking a close look at the 'pass ordering' problem 
hiding in this RFC. That is, as written and prototyped, CollageFuseOps runs 
just before the current FuseOps so that it can 'see' rewrites which guide TVM's 
native fusion rules. However some of those rewrites are target specific, and 
some are very oriented towards TVM and would likely interfere with existing 
BYOC patterns and custom lowering functions. So we want CollageFuseOps to run 
both before and after quite a few passes and something's got to give. I'll see 
if I can put a clear strawman in place to address this.
   


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


Reply via email to