mbaret commented on pull request #8795: URL: https://github.com/apache/tvm/pull/8795#issuecomment-910441110
Apologies missed point 1. Some of the directory decisions were made to loosely align with the VTA integration (another TVM-powered ML accelerator). In that, special 'VTA versions' of certain operators are defined in the VTA directory which are tightly coupled to the VTA target and compilation pathway. We have much the same concept for Ethos-U - Tensor Expressions which make sense for our target and pretty much nothing else. I think because of this, putting them into TOPI could cause confusion as they're treated very differently from all the other TOPI ops. I also agree with @manupa-arm that the ease of navigation between these closely related components is desirable. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
