> 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?
I think it is ok. -- 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-642690898
