From bc6c374f5bc6c17c3e9b5462f5f8c3c5a5e8a13e Mon Sep 17 00:00:00 2001 From: Gian Marco Iodice Date: Mon, 19 Oct 2020 12:49:44 +0100 Subject: COMPMID-3740: Remove OpenCL padding: CLWinogradInputTransformKernel - Remove padding requirement from the OpenCL kernels - Extend test to validate zero padding requirement Change-Id: I1ddf04eba783721858792efb08a2c97f11f7297e Signed-off-by: Gian Marco Iodice Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/4206 Tested-by: Arm Jenkins Reviewed-by: Michele Di Giorgio Comments-Addressed: Arm Jenkins --- src/core/CL/kernels/CLWinogradInputTransformKernel.cpp | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) (limited to 'src/core/CL/kernels/CLWinogradInputTransformKernel.cpp') diff --git a/src/core/CL/kernels/CLWinogradInputTransformKernel.cpp b/src/core/CL/kernels/CLWinogradInputTransformKernel.cpp index 6b1b86a777..c4c2b08a81 100644 --- a/src/core/CL/kernels/CLWinogradInputTransformKernel.cpp +++ b/src/core/CL/kernels/CLWinogradInputTransformKernel.cpp @@ -87,11 +87,6 @@ std::pair validate_and_configure_window(ITensorInfo *input, ITen AccessWindowRectangle input_access(input, -conv_info.pad_left(), -conv_info.pad_top(), num_elems_read_per_iteration_x, num_elems_read_per_iteration_y); window_changed = update_window_and_padding(win, input_access); } - else - { - AccessWindowStatic input_access(input, 0, -1, input->dimension(0), input->dimension(1) + 1); - window_changed = update_window_and_padding(win, input_access); - } Status err = (window_changed) ? ARM_COMPUTE_CREATE_ERROR(ErrorCode::RUNTIME_ERROR, "Insufficient Padding!") : Status{}; return std::make_pair(err, win); @@ -141,7 +136,7 @@ void CLWinogradInputTransformKernel::configure(const CLCompileContext &compile_c } else { - _border_size = BorderSize(1U, 0U, 1U, 0); + _border_size = BorderSize(); } // Compute the number of output tiles along the x and y direction of size "output_tile_size" -- cgit v1.2.1