jcf94 commented on pull request #7635:
URL: https://github.com/apache/tvm/pull/7635#issuecomment-799039579


   > > #7313 has introduced the tuning support for Sparse, now this PR brings 
the end to end model tuning support.
   > > cc @merrymercy @comaniac @FrozenGene @yuchaoli
   > > This PR also contains some bug fix:
   > > ```
   > > * Add a separate op strange for conv2d_nhwc, conv2d_nhwc.winograd and 
depthwise_conv2d_nhwc in ARM CPU that AutoScheduler can perform better
   > > 
   > > * Fix a dense bug in TFLite frontend, which will broken the 
LayoutRewrite process of AutoScheduler
   > > ```
   > 
   > I was just wondering whether we can divide this PR into 2, one with bug 
fix for conv2d and other one with sparse_dense auto scheduler & TFLite bug fix. 
These 2 are quite unrelated, may not be good to go in 1 PR.
   
   Yeah, I agree.


----------------------------------------------------------------
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:
[email protected]


Reply via email to