commit bc6c374f5bc6c17c3e9b5462f5f8c3c5a5e8a13e
author Gian Marco Iodice <gianmarco.iodice@arm.com> Mon Oct 19 12:49:44 2020 +0100
committer Gian Marco Iodice <gianmarco.iodice@arm.com> Mon Oct 19 15:36:57 2020 +0000
tree 054f99e6d8e6c266e2bfd8786b8d07ee2ef60587
parent 7333e1f10f5da9dc67b511d326121a843771a107

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 <gianmarco.iodice@arm.com>
Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/4206
Tested-by: Arm Jenkins <bsgcomp@arm.com>
Reviewed-by: Michele Di Giorgio <michele.digiorgio@arm.com>
Comments-Addressed: Arm Jenkins <bsgcomp@arm.com>

src/core/CL/kernels/CLWinogradInputTransformKernel.cpp

diff --git a/src/core/CL/kernels/CLWinogradInputTransformKernel.cpp b/src/core/CL/kernels/CLWinogradInputTransformKernel.cpp
index 6b1b86a..c4c2b08 100644
--- a/src/core/CL/kernels/CLWinogradInputTransformKernel.cpp
+++ b/src/core/CL/kernels/CLWinogradInputTransformKernel.cpp
@@ -87,11 +87,6 @@
         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 @@
     }
     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"