leandron commented on pull request #7304: URL: https://github.com/apache/tvm/pull/7304#issuecomment-781944239
> Overall LGTM. Two left comments: > > 1. The function naming. Updated the names for the ones suggested here. > 2. Clarify the status of adding TensorRT. I'd really prefer to poke someone (maybe @trevor-m) to add TensorRT on a separate patch - see my notes about the default values on the partition function in https://github.com/apache/tvm/pull/7304#discussion_r576770049. There is also vitis-ai that comes to mind, for example, we could poke @jtuyls to see whether there is interest in adding it on a separate patch, once the mechanism here is merged. @comaniac can you also have a look on the patch again, when you have some time? ---------------------------------------------------------------- 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]
