AndrewZhaoLuo commented on PR #14167: URL: https://github.com/apache/tvm/pull/14167#issuecomment-1472819633
cc @tkonolige @vinx13 This is now ready for re-review. There is a lint error because our version of `black` is out of date which might take a while to fix unfortunately (since have to update CI). Main thing was handling implicit reshapes in the layout transform (e.g. NCHW --> NCHW4c) and adding tests. Tests are composed off some manual cases + some autogenerated cases. The autogenerated cases also try to fuse compatible ops into layout transform task and mainly checks things for correctness. I offline tested ~8000 autogenerated cases for correctness of the schedule, for normal runs it tests ~9 autogenerated cases which takes about a minute on my computer. -- 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]
