COMPMID-864 Window::collapse_if_possible() is misused in several CL kernels
Removed unnecessary collapse_if_possible() calls.
Change-Id: I6f3434bc4a26470c4de5bac4e3d90b4b019c2c9c
Reviewed-on: https://eu-gerrit-1.euhpc.arm.com/117993
Reviewed-by: Michalis Spyrou <michalis.spyrou@arm.com>
Tested-by: Jenkins <bsgcomp@arm.com>
Reviewed-by: Georgios Pinitas <georgios.pinitas@arm.com>
diff --git a/src/core/CL/kernels/CLMinMaxLayerKernel.cpp b/src/core/CL/kernels/CLMinMaxLayerKernel.cpp
index 9b4533b..8ba1f77 100644
--- a/src/core/CL/kernels/CLMinMaxLayerKernel.cpp
+++ b/src/core/CL/kernels/CLMinMaxLayerKernel.cpp
@@ -1,5 +1,5 @@
/*
- * Copyright (c) 2017 ARM Limited.
+ * Copyright (c) 2017-2018 ARM Limited.
*
* SPDX-License-Identifier: MIT
*
@@ -89,7 +89,6 @@
Window window_output;
window_output.use_tensor_dimensions(_output->info()->tensor_shape());
window_output.set(Window::DimX, Window::Dimension(0, 1, 1));
- window_output.collapse_if_possible(ICLKernel::window(), 1);
Iterator output(_output, window_output);
@@ -110,27 +109,21 @@
ARM_COMPUTE_ERROR_ON_UNCONFIGURED_KERNEL(this);
ARM_COMPUTE_ERROR_ON_INVALID_SUBWINDOW(IKernel::window(), window);
- // Collapse min/max batches
Window window_collapsed = window.collapse_if_possible(ICLKernel::window(), 3);
Window slice = window_collapsed.first_slice_window_3D();
slice.set(Window::DimX, Window::Dimension(0, 1, 1));
slice.set(Window::DimY, Window::Dimension(0, 1, 1));
slice.set(Window::DimZ, Window::Dimension(0, 1, 1));
- Window window_output;
- window_output.use_tensor_dimensions(_output->info()->tensor_shape());
- window_output.set(Window::DimX, Window::Dimension(0, 1, 1));
- window_output.collapse_if_possible(ICLKernel::window(), 1);
-
- Window output_slice = window_output.first_slice_window_1D();
-
do
{
+ Window output_slice = slice.shift_dimensions(2);
+
unsigned int idx = 0;
// Set inputs
add_3D_tensor_argument(idx, _input, slice);
add_1D_tensor_argument(idx, _output, output_slice);
enqueue(queue, *this, slice);
}
- while(window.slide_window_slice_3D(slice) && window_output.slide_window_slice_1D(output_slice));
+ while(window_collapsed.slide_window_slice_3D(slice));
}