areusch commented on pull request #7250: URL: https://github.com/apache/tvm/pull/7250#issuecomment-758308337
yes also to be clear I'm very open to other suggestions...it just seems like if we are keeping this top-level API option checked-in when it only works on metal and maybe vta, we should either add detection for layout transforms or only enable it for those targets or maybe move it into e.g. a Runner subclass. I do think we should avoid putting this as a kwarg on the generic Runner since it doesn't work for many cases. ---------------------------------------------------------------- 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]
