commit 7786f2590933b8140c9e0432de5cf46b6b90a498
author Pablo Tello <pablo.tello@arm.com> Mon Aug 05 12:22:37 2019 +0100
committer Pablo Marquez <pablo.tello@arm.com> Tue Aug 06 12:09:13 2019 +0000
tree ab17635f3300bd41a3faf05522259ac493079fb0
parent a7acb3cbabeb66ce647684466a04c96b2963c9c9

COMPMID-2526: Disabled Winograd when padding > half_kernel_size

Change-Id: Id7b2d5dd4a0af2f914ee2bb35f27c07a43e2baca
Signed-off-by: Pablo Tello <pablo.tello@arm.com>
Reviewed-on: https://review.mlplatform.org/c/1688
Tested-by: Arm Jenkins <bsgcomp@arm.com>
Reviewed-by: Gian Marco Iodice <gianmarco.iodice@arm.com>

src/runtime/CL/functions/CLWinogradConvolutionLayer.cpp

diff --git a/src/runtime/CL/functions/CLWinogradConvolutionLayer.cpp b/src/runtime/CL/functions/CLWinogradConvolutionLayer.cpp
index 3b44c22..a5db977 100644
--- a/src/runtime/CL/functions/CLWinogradConvolutionLayer.cpp
+++ b/src/runtime/CL/functions/CLWinogradConvolutionLayer.cpp
@@ -158,6 +158,9 @@
     const Size2D kernel_size = Size2D(weights->tensor_shape()[idx_width], weights->tensor_shape()[idx_height]);
     const Size2D output_tile = winograd_output_tile(input_dims, kernel_size, input->data_layout());
 
+    ARM_COMPUTE_RETURN_ERROR_ON_MSG(((conv_info.pad_left() > (kernel_size.x() / 2u)) || (conv_info.pad_right() > (kernel_size.x() / 2u))), "Winograd only supports padding up to half kernel size");
+    ARM_COMPUTE_RETURN_ERROR_ON_MSG(((conv_info.pad_top() > (kernel_size.y() / 2u)) || (conv_info.pad_bottom() > (kernel_size.y() / 2u))), "Winograd only supports padding up to half kernel size");
+
     // Check if the Winograd configuration requires fast math
     if(!enable_fast_math)
     {