FrozenGene opened a new pull request #4803: [Frontend] Asymmetric padding of convolution support URL: https://github.com/apache/incubator-tvm/pull/4803 As we have supported asymmetric padding, we could avoid extra `pad` operator in the frontend now. Our tf frontend has done this. This pr complete this support in the tflite / coreml / keras frontend. Note: This is not conflict with pr https://github.com/apache/incubator-tvm/pull/4787, which could handle it and legalize it in the topi. Like our MXNet frontend could still keep 2D padding if it doesn't have asymmetric padding. However, like tflite / tf / keras / coreml has asymmetric padding, this pr could make us no extra `pad` operator now and could have better performance described in the rfc https://github.com/apache/incubator-tvm/issues/2682 @inadob @optima2005 @wyc-ruiker @anijain2305 @cchung100m Could you help to review this? Thanks.
---------------------------------------------------------------- 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] With regards, Apache Git Services
