zhuwenxi commented on pull request #47: URL: https://github.com/apache/tvm-rfcs/pull/47#issuecomment-996412022
> Thanks for the RFC and overall LGTM. One suggestion I have is that it would be great if you could provide an upstream plan that briefly explains how would you send PRs. To facilitate the review process, it is encouraged to break down your implementation to a series of small PRs. Here is an example of a PR series: > > 1. Add libxsmm to the TVM CI. > 2. Add libxsmm to TOPI. > 3. Add libxsmm to Relay op strategy. > 4. Add libxsmm to BYOC. Upstream plan added, at the end of RFC. By the way, I'm not quite sure what "TVM CI" refers to? If it means unit tests, they will be included in their related PRs when I upstream my code. -- 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]
