jiangjiajun commented on pull request #9126:
URL: https://github.com/apache/tvm/pull/9126#issuecomment-930119644


   > This PR is still too big I think considering most work here is unrelated 
to each other. Can you remove operators from this PR until you are down to 
~+300 loc?
   > 
   > Just so you know, all ops above `convert_fill_constant` I have taken a 
look at so if you reduce this PR down to those changes only the review process 
can go a lot faster
   
   Hi, @AndrewZhaoLuo 
   All the modifications are removed under `convert_fill_constant`.  Still lack 
of  lots of pull requests to finish my work, I'll try to classify these pull 
request to make reviewing faster


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


Reply via email to