junrushao1994 commented on pull request #7153: URL: https://github.com/apache/tvm/pull/7153#issuecomment-800440041
It is super cool work that has been missing for years. It looks really nice and I don't have other concerns besides the ExternalProject approach, where the internet is required on the first build, which I (being nitpicky) think could cause some issues if the build environment doesn't have network connection. A better approach might be creating a repo under tlc-pack/octoml/etc and submodule it, like what synr does. ---------------------------------------------------------------- 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]
