From 7786f2590933b8140c9e0432de5cf46b6b90a498 Mon Sep 17 00:00:00 2001 From: Pablo Tello Date: Mon, 5 Aug 2019 12:22:37 +0100 Subject: COMPMID-2526: Disabled Winograd when padding > half_kernel_size Change-Id: Id7b2d5dd4a0af2f914ee2bb35f27c07a43e2baca Signed-off-by: Pablo Tello Reviewed-on: https://review.mlplatform.org/c/1688 Tested-by: Arm Jenkins Reviewed-by: Gian Marco Iodice --- src/runtime/CL/functions/CLWinogradConvolutionLayer.cpp | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/runtime/CL/functions/CLWinogradConvolutionLayer.cpp b/src/runtime/CL/functions/CLWinogradConvolutionLayer.cpp index 3b44c22a27..a5db977371 100644 --- a/src/runtime/CL/functions/CLWinogradConvolutionLayer.cpp +++ b/src/runtime/CL/functions/CLWinogradConvolutionLayer.cpp @@ -158,6 +158,9 @@ Status CLWinogradConvolutionLayer::validate(const ITensorInfo *input, const ITen 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) { -- cgit v1.2.1