tmoreau89 commented on pull request #5842:
URL: https://github.com/apache/incubator-tvm/pull/5842#issuecomment-660355629


   @zhanghaohit I took a second look at the changes in this PR, and I believe 
it would be best to break the PR down into separate, smaller PRs:
   
   (1) modifications to VTA for OpenCL support (runtime, operator support etc.)
   (2) quantization support for conv2d_transpose which is device agnostic
   (3) modifications to device_annotation.cc
   (4) once (3) is merged and approved, updating GraphPack pass to use device 
annotation
   
   Let me know what you think. This will make the PRs easier to review and 
merge.


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

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


Reply via email to