[GitHub] [tvm] zhiics edited a comment on pull request #7060: [WIP] Add MLIR Relay Dialect, Translate MLIR HLO to MLIR Relay to Relay

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

zhiics edited a comment 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 MLIR/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:
users@infra.apache.org