[GitHub] [incubator-tvm] jroesch commented on pull request #6797: [TVMSCRIPT] Using diagnostics for TVM Script

[GitBox <gi...@apache.org>]

jroesch commented on pull request #6797:
URL: https://github.com/apache/incubator-tvm/pull/6797#issuecomment-721992410


   > Do we prefer to maintain synr as an external dependency, and make a release every time, or include synr as part of tvm?
   
   I think it is valuable to maintain outside of the TVM code base. I know monorepo'ing everything provides some conveniences but when there are reusable components which can be used outside of TVM it makes sense to maintain them as separate pieces of code. For example there have been multiple other projects which have built similar approaches to `synr` and I think it is valuable to provide it in an easily reusable component, for example TensorFlow and PyTorch have both opt'd not to do this and even though they do AST rewriting there is no easy way to take advantage of all the work they did. 


----------------------------------------------------------------
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:
users@infra.apache.org