You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tvm.apache.org by GitBox <gi...@apache.org> on 2022/09/02 16:51:53 UTC

[GitHub] [tvm] guberti commented on a diff in pull request #12448: Add Arm DSP implementation of Depthwise Conv2D

guberti commented on code in PR #12448:
URL: https://github.com/apache/tvm/pull/12448#discussion_r961859675


##########
python/tvm/relay/op/strategy/arm_cpu.py:
##########
@@ -235,6 +235,28 @@ def conv2d_strategy_arm_cpu(attrs, inputs, out_type, target):
                     wrap_topi_schedule(topi.arm_cpu.schedule_depthwise_conv2d_nhwc),
                     name="depthwise_conv2d_nhwc.arm_cpu",
                 )
+
+            # Optimized special case depthwiseConv2D operation. Requires a 3x3 kernel, a
+            # NHWC layout, a HWOI kernel layout (which we would ideally rearrange), no dilation,
+            # "SAME" padding, int8 inputs and outputs, the same number of input and output

Review Comment:
   Fixed! The schedule supports any form of padding - SAME, VALID, or a custom tuple.



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

To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org

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