zhiics commented on pull request #7060: URL: https://github.com/apache/tvm/pull/7060#issuecomment-741866746
Thanks @yongwww for the great effort:) Both paths make sense to me. Adding Relay as a dialect would benefit more other dialects including other projects internally. But it does introduce another layer of transition for TVM. Since converting through MLIR/HLO to Relay shares almost the same amount of efforts, we can probably go this route first. We could either add relay as a dialect later after op schema is done as @tqchen mentioned or go through a dialect to HLO then to Relay first if it is needed. As of now, we probably don't need many dialects to Relay, i.e. we have directly parsers for onnx/TFLite to Relay. If you agree, let's discuss offline to setup the CI so that you can move forward. ---------------------------------------------------------------- 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: us...@infra.apache.org
