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/08/20 22:41:38 UTC

[GitHub] [tvm] jroesch commented on pull request #8806: [3/6] Arm(R) Ethos(TM)-U NPU TIR passes and TE for Conv2D

jroesch commented on pull request #8806:
URL: https://github.com/apache/tvm/pull/8806#issuecomment-902993040


   Thanks for the contribution guys!
   
   I know how painful of a request this is, but is there any way you guys could logically split this up into smaller sets of changes? I know I am one of the worst offenders of very large PRs but an 8k PR is very hard/nearly impossible to review well in one shot. 
   
   My hope is for us to build wider understanding of these big changes so we can spread the future review and maintenance burden across many people. For example @areusch has the most context but is on vacation for the next couple of weeks. 
   
   


-- 
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