Hi @FrozenGene , 
I agree that different strategies should be available to the auto-tuner. See if 
the solution proposed is good enough for you (at least as a temporary 
work-around). For Armv7-A or NCHW, nothing changes, we follow exactly the 
previous path. 

For Armv8-A and NHWC we don't convert during the legalization step, but during 
the `_alter_conv2d_layout` pass. The only difference now is that the offset 
contribution will be added after the convolution instead than before. 

I agree that a better solution, where the legalization changes depending on the 
strategy, would be better. However, I don't think the legalization step has got 
enough information to know the strategy (for now). 

What do you think?

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/apache/incubator-tvm/pull/5754#issuecomment-642682834

Reply via email to