You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@tvm.apache.org by GitBox <gi...@apache.org> on 2021/12/17 03:41:16 UTC

[GitHub] [tvm-rfcs] zhuwenxi commented on pull request #47: RFC to integrate LIBXSMM with TVM.

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: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org