commit b158fba3d2724abab70706f04069da6fc9146429
author Georgios Pinitas <georgios.pinitas@arm.com> Wed Jul 05 16:50:24 2017 +0100
committer Anthony Barbier <anthony.barbier@arm.com> Mon Sep 17 14:15:39 2018 +0100
tree 04a7e0f9cf607f565b3857c211efed92dc702005
parent 9502f31df35f3006c2e3d9b67d1d586e02646d75

COMPMID-417: Fix output access window in ChannelExtract Kernels.

Change-Id: I0349ef7205e316d85a01e83e86016310143f8886
Reviewed-on: http://mpd-gerrit.cambridge.arm.com/79820
Tested-by: Kaizen <jeremy.johnson+kaizengerrit@arm.com>
Reviewed-by: Moritz Pflanzer <moritz.pflanzer@arm.com>

src/core/NEON/kernels/NEChannelExtractKernel.cpp

diff --git a/src/core/NEON/kernels/NEChannelExtractKernel.cpp b/src/core/NEON/kernels/NEChannelExtractKernel.cpp
index ebc4b85..dc31685 100644
--- a/src/core/NEON/kernels/NEChannelExtractKernel.cpp
+++ b/src/core/NEON/kernels/NEChannelExtractKernel.cpp
@@ -148,12 +148,11 @@
     _input  = input;
     _output = output;
 
-    Window                win = calculate_max_window(*input->info(), Steps(num_elems_processed_per_iteration));
-    AccessWindowRectangle output_access(input->info(), 0, 0, num_elems_processed_per_iteration, 1, 1.f / subsampling, 1.f / subsampling);
+    Window                 win = calculate_max_window(*input->info(), Steps(num_elems_processed_per_iteration));
+    AccessWindowHorizontal input_access(input->info(), 0, num_elems_processed_per_iteration);
+    AccessWindowRectangle  output_access(output->info(), 0, 0, num_elems_processed_per_iteration, 1, 1.f / subsampling, 1.f / subsampling);
 
-    update_window_and_padding(win,
-                              AccessWindowHorizontal(input->info(), 0, num_elems_processed_per_iteration),
-                              output_access);
+    update_window_and_padding(win, input_access, output_access);
 
     ValidRegion input_valid_region = input->info()->valid_region();
 
@@ -257,8 +256,8 @@
 
     _output                    = output;
     Window                 win = calculate_max_window(*_input->info(), Steps(num_elems_processed_per_iteration));
-    AccessWindowHorizontal output_access(output->info(), 0, num_elems_processed_per_iteration);
     AccessWindowHorizontal input_access(_input->info(), 0, num_elems_processed_per_iteration);
+    AccessWindowHorizontal output_access(output->info(), 0, num_elems_processed_per_iteration);
     update_window_and_padding(win, input_access, output_access);
     output_access.set_valid_region(win, _input->info()->valid_region());