From c1a72451273ec019e3e74c4b53ea847afe8ddf7c Mon Sep 17 00:00:00 2001 From: Georgios Pinitas Date: Fri, 24 Aug 2018 11:25:32 +0100 Subject: COMPMID-1332: Implement Slice for CL Change-Id: I0dbc4fd7f640d31daa1970eb3da0e941cb771f2b Reviewed-on: https://eu-gerrit-1.euhpc.arm.com/146145 Tested-by: Jenkins Reviewed-by: Giorgio Arena Reviewed-by: Michalis Spyrou --- arm_compute/core/utils/helpers/tensor_transform.h | 32 ++- arm_compute/runtime/CL/CLFunctions.h | 1 + arm_compute/runtime/CL/functions/CLSlice.h | 69 +++++ src/core/CL/CLKernelLibrary.cpp | 6 +- src/core/CL/cl_kernels/slice_ops.cl | 107 ++++++++ src/core/CL/cl_kernels/strided_slice.cl | 85 ------ src/core/CL/kernels/CLStridedSliceKernel.cpp | 21 +- src/core/utils/helpers/tensor_transform.cpp | 24 ++ src/runtime/CL/functions/CLSlice.cpp | 61 +++++ tests/datasets/SliceOperationsDataset.h | 286 +++++++++++++++++++++ tests/datasets/StridedSliceDataset.h | 181 ------------- tests/validation/CL/Slice.cpp | 133 ++++++++++ tests/validation/CL/StridedSlice.cpp | 4 +- .../validation/fixtures/SliceOperationsFixtures.h | 182 +++++++++++++ tests/validation/fixtures/StridedSliceFixture.h | 119 --------- tests/validation/reference/SliceOperations.cpp | 137 ++++++++++ tests/validation/reference/SliceOperations.h | 48 ++++ tests/validation/reference/StridedSlice.cpp | 92 ------- tests/validation/reference/StridedSlice.h | 45 ---- 19 files changed, 1100 insertions(+), 533 deletions(-) create mode 100644 arm_compute/runtime/CL/functions/CLSlice.h create mode 100644 src/core/CL/cl_kernels/slice_ops.cl delete mode 100644 src/core/CL/cl_kernels/strided_slice.cl create mode 100644 src/runtime/CL/functions/CLSlice.cpp create mode 100644 tests/datasets/SliceOperationsDataset.h delete mode 100644 tests/datasets/StridedSliceDataset.h create mode 100644 tests/validation/CL/Slice.cpp create mode 100644 tests/validation/fixtures/SliceOperationsFixtures.h delete mode 100644 tests/validation/fixtures/StridedSliceFixture.h create mode 100644 tests/validation/reference/SliceOperations.cpp create mode 100644 tests/validation/reference/SliceOperations.h delete mode 100644 tests/validation/reference/StridedSlice.cpp delete mode 100644 tests/validation/reference/StridedSlice.h diff --git a/arm_compute/core/utils/helpers/tensor_transform.h b/arm_compute/core/utils/helpers/tensor_transform.h index e5b32a5372..966c1f1fdf 100644 --- a/arm_compute/core/utils/helpers/tensor_transform.h +++ b/arm_compute/core/utils/helpers/tensor_transform.h @@ -32,12 +32,33 @@ namespace helpers { namespace tensor_transform { +/** Returns the absolute ends coordinates of slice + * + * @param[in] input_shape Input tensor shape + * @param[in] ends End coordinates + * + * @return Absolute end coordinate + */ +Coordinates slice_absolute_end_coords(TensorShape input_shape, Coordinates ends); + +/** Computes output shape of slice + * + * @warning Ends must be non-negative + * + * @param[in] input_shape Input tensor shape + * @param[in] starts Start coordinates + * @param[in] ends_abs Absolute end coordinates + * + * @return The output tensor shape + */ +TensorShape compute_slice_output_shape(TensorShape input_shape, Coordinates starts, Coordinates ends_abs); + /** Returns the absolute start coordinates of strided slice * * @param[in] input_shape Input tensor shape * @param[in] starts Start coordinates * @param[in] strides Slice strides - * @param[in] begin_mask (Optional) If the ith bit of begin_mask is set, begin[i] is ignored and + * @param[in] begin_mask (Optional) If the ith bit of begin_mask is set, starts[i] is ignored and * the fullest possible range in that dimension is used instead. * * @return Absolute start coordinates @@ -45,6 +66,8 @@ namespace tensor_transform Coordinates strided_slice_absolute_start_coords(TensorShape input_shape, Coordinates starts, Coordinates strides, int32_t begin_mask = 0); /** Returns the absolute ends coordinates of strided slice + * + * @warning Starts must be non-negative * * @param[in] input_shape Input tensor shape * @param[in] starts_abs Absolute start coordinates @@ -53,7 +76,7 @@ Coordinates strided_slice_absolute_start_coords(TensorShape input_shape, Coordin * @param[in] end_mask (Optional) If the ith bit of end_mask is set, end[i] is ignored and * the fullest possible range in that dimension is used instead. * @param[in] shrink_axis_mask (Optional) If the ith bit of shrink_axis_mask is set, it implies that the ith specification shrinks the dimensionality by 1. - * A slice of size 1 starting from begin[i] in the dimension must be preserved. + * A slice of size 1 starting from starts[i] in the dimension must be preserved. * * @return Absolute end coordinates */ @@ -68,7 +91,10 @@ Coordinates strided_slice_absolute_end_coords(TensorShape input_shape, Coordinat */ Coordinates strided_slice_strides(TensorShape input_shape, Coordinates strides); -/** Computes output shape of a strided slice +/** Computes output shape of strided slice + * + * @warning Starts and ends must be non-negative + * @warning Starts, ends and final strides should have the same dimensions as the input shape * * @param[in] input_shape Input tensor shape * @param[in] starts_abs Absolute start coordinates diff --git a/arm_compute/runtime/CL/CLFunctions.h b/arm_compute/runtime/CL/CLFunctions.h index 76c2128f2c..d2bfdfd7cb 100644 --- a/arm_compute/runtime/CL/CLFunctions.h +++ b/arm_compute/runtime/CL/CLFunctions.h @@ -106,6 +106,7 @@ #include "arm_compute/runtime/CL/functions/CLReshapeLayer.h" #include "arm_compute/runtime/CL/functions/CLScale.h" #include "arm_compute/runtime/CL/functions/CLScharr3x3.h" +#include "arm_compute/runtime/CL/functions/CLSlice.h" #include "arm_compute/runtime/CL/functions/CLSobel3x3.h" #include "arm_compute/runtime/CL/functions/CLSobel5x5.h" #include "arm_compute/runtime/CL/functions/CLSobel7x7.h" diff --git a/arm_compute/runtime/CL/functions/CLSlice.h b/arm_compute/runtime/CL/functions/CLSlice.h new file mode 100644 index 0000000000..acd4f0d3ad --- /dev/null +++ b/arm_compute/runtime/CL/functions/CLSlice.h @@ -0,0 +1,69 @@ +/* + * Copyright (c) 2018 ARM Limited. + * + * SPDX-License-Identifier: MIT + * + * Permission is hereby granted, free of charge, to any person obtaining a copy + * of this software and associated documentation files (the "Software"), to + * deal in the Software without restriction, including without limitation the + * rights to use, copy, modify, merge, publish, distribute, sublicense, and/or + * sell copies of the Software, and to permit persons to whom the Software is + * furnished to do so, subject to the following conditions: + * + * The above copyright notice and this permission notice shall be included in all + * copies or substantial portions of the Software. + * + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE + * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE + * SOFTWARE. + */ +#ifndef __ARM_COMPUTE_CL_SLICE_H__ +#define __ARM_COMPUTE_CL_SLICE_H__ + +#include "arm_compute/runtime/CL/ICLSimpleFunction.h" + +namespace arm_compute +{ +// Forward Declarations +class ICLTensor; + +/** Basic function to perform tensor slicing */ +class CLSlice : public ICLSimpleFunction +{ +public: + /** Configure kernel + * + * @note Supported tensor rank: up to 4 + * @note Start indices must be non-negative. 0 <= starts[i] + * @note End coordinates can be negative, which represents the number of elements before the end of that dimension. + * @note End indices are not inclusive unless negative. + * + * @param[in] input Source tensor. Data type supported: U8/S8/QASYMM8/U16/S16/U32/S32/F16/F32 + * @param[out] output Destination tensor. Data type supported: Same as @p input + * @param[in] starts The starts of the dimensions of the input tensor to be sliced. The length must be of rank(input). + * @param[in] ends The ends of the dimensions of the input tensor to be sliced. The length must be of rank(input). + */ + void configure(const ICLTensor *input, ICLTensor *output, const Coordinates &starts, const Coordinates &ends); + + /** Static function to check if given info will lead to a valid configuration of @ref CLSlice + * + * @note Supported tensor rank: up to 4 + * @note Start indices must be non-negative. 0 <= starts[i] + * @note End coordinates can be negative, which represents the number of elements before the end of that dimension. + * @note End indices are not inclusive unless negative. + * + * @param[in] input Source tensor info. Data type supported: U8/S8/QASYMM8/U16/S16/U32/S32/F16/F32 + * @param[in] output Destination tensor info. Data type supported: Same as @p input + * @param[in] starts The starts of the dimensions of the input tensor to be sliced. The length must be of rank(input). + * @param[in] ends The ends of the dimensions of the input tensor to be sliced. The length must be of rank(input). + * + * @return A status + */ + static Status validate(const ITensorInfo *input, const ITensorInfo *output, const Coordinates &starts, const Coordinates &ends); +}; +} // namespace arm_compute +#endif /* __ARM_COMPUTE_CL_SLICE_H__ */ diff --git a/src/core/CL/CLKernelLibrary.cpp b/src/core/CL/CLKernelLibrary.cpp index 29fd672a96..0cc6e320bf 100644 --- a/src/core/CL/CLKernelLibrary.cpp +++ b/src/core/CL/CLKernelLibrary.cpp @@ -361,7 +361,7 @@ const std::map CLKernelLibrary::_kernel_program_map = { "softmax_layer_max_shift_exp_sum_quantized_parallel", "softmax_layer_quantized.cl" }, { "softmax_layer_max_shift_exp_sum_serial", "softmax_layer.cl" }, { "softmax_layer_max_shift_exp_sum_parallel", "softmax_layer.cl" }, - { "strided_slice", "strided_slice.cl" }, + { "strided_slice", "slice_ops.cl" }, { "suppress_non_maximum", "canny.cl" }, { "tablelookup_U8", "tablelookup.cl" }, { "tablelookup_S16", "tablelookup.cl" }, @@ -742,8 +742,8 @@ const std::map CLKernelLibrary::_program_source_map = #include "./cl_kernels/softmax_layer_quantized.clembed" }, { - "strided_slice.cl", -#include "./cl_kernels/strided_slice.clembed" + "slice_ops.cl", +#include "./cl_kernels/slice_ops.clembed" }, { "tablelookup.cl", diff --git a/src/core/CL/cl_kernels/slice_ops.cl b/src/core/CL/cl_kernels/slice_ops.cl new file mode 100644 index 0000000000..bc3df47345 --- /dev/null +++ b/src/core/CL/cl_kernels/slice_ops.cl @@ -0,0 +1,107 @@ +/* + * Copyright (c) 2018 ARM Limited. + * + * SPDX-License-Identifier: MIT + * + * Permission is hereby granted, free of charge, to any person obtaining a copy + * of this software and associated documentation files (the "Software"), to + * deal in the Software without restriction, including without limitation the + * rights to use, copy, modify, merge, publish, distribute, sublicense, and/or + * sell copies of the Software, and to permit persons to whom the Software is + * furnished to do so, subject to the following conditions: + * + * The above copyright notice and this permission notice shall be included in all + * copies or substantial portions of the Software. + * + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE + * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE + * SOFTWARE. + */ +#include "helpers.h" + +/** Perform a strided slice operation on a given input. + * + * @attention Supported tensor rank: up to 4 + * + * @attention Data type can be passed using the -DDATA_TYPE compile flag, e.g. -DDATA_TYPE=float + * @attention Input and output tensor dephts should be given as a preprocessor arguments using -DSRC_DEPTH=size. and -DDST_DEPTH=size + * @attention Absolute start coordinates for each dimension should be given as preprocessor -DSTART_index=value e.g. -DSTART_0=2 + * @attention Strides for each dimension should be given as preprocessor -DSTRIDE_index=value e.g. -DSTRIDE_1=1 + * + * @param[in] input_ptr Pointer to the source tensor. Supported data types: U8/S8/QASYMM8/U16/S16/F16/U32/S32/F32 + * @param[in] input_stride_x Stride of the source tensor in X dimension (in bytes) + * @param[in] input_step_x input_stride_x * number of elements along X processed per workitem(in bytes) + * @param[in] input_stride_y Stride of the source tensor in Y dimension (in bytes) + * @param[in] input_step_y input_stride_y * number of elements along Y processed per workitem(in bytes) + * @param[in] input_stride_z Stride of the source tensor in Z dimension (in bytes) + * @param[in] input_step_z input_stride_z * number of elements along Z processed per workitem(in bytes) + * @param[in] input_stride_w Stride of the source tensor in W dimension (in bytes) + * @param[in] input_step_w input_stride_w * number of elements along W processed per workitem(in bytes) + * @param[in] input_offset_first_element_in_bytes The offset of the first element in the source tensor + * @param[out] output_ptr Pointer to the destination tensor. Supported data types: same as @p input_ptr + * @param[in] output_stride_x Stride of the destination tensor in X dimension (in bytes) + * @param[in] output_step_x output_stride_x * number of elements along X processed per workitem(in bytes) + * @param[in] output_stride_y Stride of the destination tensor in Y dimension (in bytes) + * @param[in] output_step_y output_stride_y * number of elements along Y processed per workitem(in bytes) + * @param[in] output_stride_z Stride of the destination tensor in Z dimension (in bytes) + * @param[in] output_step_z output_stride_z * number of elements along Z processed per workitem(in bytes) + * @param[in] output_stride_w Stride of the destination tensor in W dimension (in bytes) + * @param[in] output_step_w output_stride_w * number of elements along W processed per workitem(in bytes) + * @param[in] output_offset_first_element_in_bytes The offset of the first element in the destination tensor + */ +__kernel void strided_slice( + TENSOR4D_DECLARATION(input), + TENSOR4D_DECLARATION(output)) +{ + // Get pixels pointer + Tensor4D input = CONVERT_TO_TENSOR4D_STRUCT_NO_STEP(input, SRC_DEPTH); + Tensor4D output = CONVERT_TO_TENSOR4D_STRUCT(output, DST_DEPTH); + + int offset = 0; + + // Offset X +#if defined(START_0) && defined(STRIDE_0) && defined(VEC_SIZE) && defined(LAST_ACCESSED_X) + // Check if access on width gets out of bounds + // If it does shift access vector to access elements within bounds + const int xi = (int)(get_global_id(0) * VEC_SIZE); + offset = (int)START_0 + min(xi, (int)LAST_ACCESSED_X); + input.ptr += offset * input_stride_x; + output.ptr -= max(xi - (int)LAST_ACCESSED_X, 0) * output_stride_x; +#elif defined(START_0) && defined(STRIDE_0) + offset = (int)START_0 + (int)get_global_id(0) * (int)STRIDE_0; + input.ptr += offset * input_stride_x; +#endif // defined(START_0) && defined(STRIDE_0) + + // Offset Y +#if defined(START_1) && defined(STRIDE_1) + offset = (int)START_1 + (int)get_global_id(1) * (int)STRIDE_1; + input.ptr += offset * input_stride_y; +#endif // defined(START_1) && defined(STRIDE_1) + + // Offset Z +#if defined(START_2) && defined(STRIDE_2) + offset = (int)START_2 + ((int)get_global_id(2) % (int)DST_DEPTH) * (int)STRIDE_2; + input.ptr += offset * input_stride_z; +#endif // defined(START_2) && defined(STRIDE_2) + + // Offset depth +#if defined(START_3) && defined(STRIDE_3) + offset = (int)START_3 + ((int)get_global_id(2) / (int)DST_DEPTH) * (int)STRIDE_3; + input.ptr += offset * input_stride_w; +#endif // defined(START_3) && defined(STRIDE_3) + + // Store result +#if defined(VEC_SIZE) && defined(LAST_ACCESSED_X) + VEC_DATA_TYPE(DATA_TYPE, VEC_SIZE) + val = VLOAD(VEC_SIZE)(0, (__global DATA_TYPE *)(input.ptr)); + + VSTORE(VEC_SIZE) + (val, 0, (__global DATA_TYPE *)(output.ptr)); +#else // defined(VEC_SIZE) && defined(LAST_ACCESSED_X) + *((__global DATA_TYPE *)(output.ptr)) = *((__global DATA_TYPE *)(input.ptr)); +#endif // defined(VEC_SIZE) && defined(LAST_ACCESSED_X) +} diff --git a/src/core/CL/cl_kernels/strided_slice.cl b/src/core/CL/cl_kernels/strided_slice.cl deleted file mode 100644 index 7c68fb9a07..0000000000 --- a/src/core/CL/cl_kernels/strided_slice.cl +++ /dev/null @@ -1,85 +0,0 @@ -/* - * Copyright (c) 2018 ARM Limited. - * - * SPDX-License-Identifier: MIT - * - * Permission is hereby granted, free of charge, to any person obtaining a copy - * of this software and associated documentation files (the "Software"), to - * deal in the Software without restriction, including without limitation the - * rights to use, copy, modify, merge, publish, distribute, sublicense, and/or - * sell copies of the Software, and to permit persons to whom the Software is - * furnished to do so, subject to the following conditions: - * - * The above copyright notice and this permission notice shall be included in all - * copies or substantial portions of the Software. - * - * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR - * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, - * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE - * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER - * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, - * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE - * SOFTWARE. - */ -#include "helpers.h" - -/** Perform a strided slice operation on a given input. - * - * @attention Supported tensor rank: up to 4 - * - * @attention Data type can be passed using the -DDATA_TYPE compile flag, e.g. -DDATA_TYPE=float - * @attention Input and output tensor dephts should be given as a preprocessor arguments using -DSRC_DEPTH=size. and -DDST_DEPTH=size - * @attention Absolute start coordinates for each dimension should be given as preprocessor -DSTART_index=value e.g. -DSTART_0=2 - * @attention Strides for each dimension should be given as preprocessor -DSTRIDE_index=value e.g. -DSTRIDE_1=1 - * - * @param[in] input_ptr Pointer to the source tensor. Supported data types: U8/S8/QASYMM8/U16/S16/F16/U32/S32/F32 - * @param[in] input_stride_x Stride of the source tensor in X dimension (in bytes) - * @param[in] input_step_x input_stride_x * number of elements along X processed per workitem(in bytes) - * @param[in] input_stride_y Stride of the source tensor in Y dimension (in bytes) - * @param[in] input_step_y input_stride_y * number of elements along Y processed per workitem(in bytes) - * @param[in] input_stride_z Stride of the source tensor in Z dimension (in bytes) - * @param[in] input_step_z input_stride_z * number of elements along Z processed per workitem(in bytes) - * @param[in] input_stride_w Stride of the source tensor in W dimension (in bytes) - * @param[in] input_step_w input_stride_w * number of elements along W processed per workitem(in bytes) - * @param[in] input_offset_first_element_in_bytes The offset of the first element in the source tensor - * @param[out] output_ptr Pointer to the destination tensor. Supported data types: same as @p input_ptr - * @param[in] output_stride_x Stride of the destination tensor in X dimension (in bytes) - * @param[in] output_step_x output_stride_x * number of elements along X processed per workitem(in bytes) - * @param[in] output_stride_y Stride of the destination tensor in Y dimension (in bytes) - * @param[in] output_step_y output_stride_y * number of elements along Y processed per workitem(in bytes) - * @param[in] output_stride_z Stride of the destination tensor in Z dimension (in bytes) - * @param[in] output_step_z output_stride_z * number of elements along Z processed per workitem(in bytes) - * @param[in] output_stride_w Stride of the destination tensor in W dimension (in bytes) - * @param[in] output_step_w output_stride_w * number of elements along W processed per workitem(in bytes) - * @param[in] output_offset_first_element_in_bytes The offset of the first element in the destination tensor - */ -__kernel void strided_slice( - TENSOR4D_DECLARATION(input), - TENSOR4D_DECLARATION(output)) -{ - // Get pixels pointer - Tensor4D input = CONVERT_TO_TENSOR4D_STRUCT_NO_STEP(input, SRC_DEPTH); - Tensor4D output = CONVERT_TO_TENSOR4D_STRUCT(output, DST_DEPTH); - - int offset_0 = 0; - int offset_1 = 0; - int offset_2 = 0; - int offset_3 = 0; - - // Calculate offset -#if defined(START_0) && defined(STRIDE_0) - offset_0 = (int)START_0 + (int)get_global_id(0) * (int)STRIDE_0; -#endif // defined(START_0) && defined(STRIDE_0) -#if defined(START_1) && defined(STRIDE_1) - offset_1 = (int)START_1 + (int)get_global_id(1) * (int)STRIDE_1; -#endif // defined(START_1) && defined(STRIDE_1) -#if defined(START_2) && defined(STRIDE_2) - offset_2 = (int)START_2 + ((int)get_global_id(2) % (int)DST_DEPTH) * (int)STRIDE_2; -#endif // defined(START_2) && defined(STRIDE_2) -#if defined(START_3) && defined(STRIDE_3) - offset_3 = (int)START_3 + ((int)get_global_id(2) / (int)DST_DEPTH) * (int)STRIDE_3; -#endif // defined(START_2) && defined(STRIDE_2) - - // Store result - *((__global DATA_TYPE *)(output.ptr)) = *((__global DATA_TYPE *)tensor4D_offset(&input, offset_0, offset_1, offset_2, offset_3)); -} diff --git a/src/core/CL/kernels/CLStridedSliceKernel.cpp b/src/core/CL/kernels/CLStridedSliceKernel.cpp index f07436ac60..2d2ba103e5 100644 --- a/src/core/CL/kernels/CLStridedSliceKernel.cpp +++ b/src/core/CL/kernels/CLStridedSliceKernel.cpp @@ -55,10 +55,10 @@ Status validate_arguments(const ITensorInfo *input, const ITensorInfo *output, ARM_COMPUTE_RETURN_ERROR_ON(starts.num_dimensions() > input->num_dimensions()); ARM_COMPUTE_RETURN_ERROR_ON(ends.num_dimensions() > input->num_dimensions()); ARM_COMPUTE_RETURN_ERROR_ON(strides.num_dimensions() > input->num_dimensions()); - for(unsigned int i = 0; i < strides.num_dimensions(); ++i) + ARM_COMPUTE_RETURN_ERROR_ON(std::any_of(strides.cbegin(), strides.cbegin() + strides.num_dimensions(), [](int i) { - ARM_COMPUTE_RETURN_ERROR_ON(strides[i] == 0); - } + return i == 0; + })); // Get expected output shape const TensorShape exp_output_shape = arm_compute::misc::shape_calculator::compute_strided_slice_shape(*input, @@ -120,6 +120,19 @@ void CLStridedSliceKernel::configure(const ICLTensor *input, ICLTensor *output, // Configure kernel window auto win_config = validate_and_configure_window(input->info(), output->info(), starts, ends, strides, begin_mask, end_mask, shrink_axis_mask); ARM_COMPUTE_ERROR_THROW_ON(win_config.first); + + // Enable multiple elements processing along x if stride_x is 1 and output width greater than the access vector size + const int vec_size_x = 16 / input->info()->element_size(); + const int output_width_x = output->info()->tensor_shape().x(); + const bool multi_access_x = (final_strides.x() == 1) && (output_width_x / vec_size_x > 0); + + // Update window if needed + if(multi_access_x) + { + Window &updated_window = std::get<1>(win_config); + updated_window.set(Window::DimX, + Window::Dimension(updated_window.x().start(), ceil_to_multiple(updated_window.x().end(), vec_size_x), vec_size_x)); + } ICLKernel::configure_internal(win_config.second); // Create build options @@ -130,6 +143,8 @@ void CLStridedSliceKernel::configure(const ICLTensor *input, ICLTensor *output, build_opts.add_option("-DSTART_" + support::cpp11::to_string(i) + "=" + support::cpp11::to_string(starts_abs[i])); build_opts.add_option("-DSTRIDE_" + support::cpp11::to_string(i) + "=" + support::cpp11::to_string(final_strides[i])); } + build_opts.add_option_if(multi_access_x, "-DLAST_ACCESSED_X=" + support::cpp11::to_string(std::max(output_width_x - vec_size_x, 0))); + build_opts.add_option_if(multi_access_x, "-DVEC_SIZE=" + support::cpp11::to_string(vec_size_x)); build_opts.add_option_if_else(input_shape.num_dimensions() > 2, "-DSRC_DEPTH=" + support::cpp11::to_string(input_shape.z()), "-DSRC_DEPTH=1"); diff --git a/src/core/utils/helpers/tensor_transform.cpp b/src/core/utils/helpers/tensor_transform.cpp index 5c83a8bdb5..a4bce5da5a 100644 --- a/src/core/utils/helpers/tensor_transform.cpp +++ b/src/core/utils/helpers/tensor_transform.cpp @@ -29,6 +29,30 @@ namespace helpers { namespace tensor_transform { +Coordinates slice_absolute_end_coords(TensorShape input_shape, Coordinates ends) +{ + // Create end mask + int32_t end_mask = 0; + for(unsigned int i = 0; i < ends.num_dimensions(); ++i) + { + if(ends[i] < 0) + { + end_mask |= 1 << i; + } + } + // Get unit strides + const BiStrides unit_strides = strided_slice_strides(input_shape, BiStrides()); + + return strided_slice_absolute_end_coords(input_shape, Coordinates(), ends, unit_strides, end_mask); +} + +TensorShape compute_slice_output_shape(TensorShape input_shape, Coordinates starts, Coordinates ends_abs) +{ + // Get unit strides + const BiStrides unit_strides = strided_slice_strides(input_shape, BiStrides()); + return compute_strided_slice_output_shape(input_shape, starts, ends_abs, unit_strides); +} + Coordinates strided_slice_absolute_start_coords(TensorShape input_shape, Coordinates starts, Coordinates strides, int32_t begin_mask) { Coordinates starts_abs; diff --git a/src/runtime/CL/functions/CLSlice.cpp b/src/runtime/CL/functions/CLSlice.cpp new file mode 100644 index 0000000000..bef7eca71c --- /dev/null +++ b/src/runtime/CL/functions/CLSlice.cpp @@ -0,0 +1,61 @@ +/* + * Copyright (c) 2018 ARM Limited. + * + * SPDX-License-Identifier: MIT + * + * Permission is hereby granted, free of charge, to any person obtaining a copy + * of this software and associated documentation files (the "Software"), to + * deal in the Software without restriction, including without limitation the + * rights to use, copy, modify, merge, publish, distribute, sublicense, and/or + * sell copies of the Software, and to permit persons to whom the Software is + * furnished to do so, subject to the following conditions: + * + * The above copyright notice and this permission notice shall be included in all + * copies or substantial portions of the Software. + * + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE + * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE + * SOFTWARE. + */ +#include "arm_compute/runtime/CL/functions/CLSlice.h" + +#include "arm_compute/core/CL/ICLTensor.h" +#include "arm_compute/core/CL/kernels/CLStridedSliceKernel.h" +#include "arm_compute/core/Types.h" +#include "arm_compute/core/utils/helpers/tensor_transform.h" +#include "support/ToolchainSupport.h" + +namespace arm_compute +{ +void CLSlice::configure(const ICLTensor *input, ICLTensor *output, const Coordinates &starts, const Coordinates &ends) +{ + ARM_COMPUTE_ERROR_ON_NULLPTR(input); + + // Get absolute end coordinates + const Coordinates ends_abs = arm_compute::helpers::tensor_transform::slice_absolute_end_coords(input->info()->tensor_shape(), ends); + + auto k = arm_compute::support::cpp14::make_unique(); + k->configure(input, output, starts, ends_abs, BiStrides(), 0, 0, 0); + _kernel = std::move(k); +} + +Status CLSlice::validate(const ITensorInfo *input, const ITensorInfo *output, const Coordinates &starts, const Coordinates &ends) +{ + ARM_COMPUTE_RETURN_ERROR_ON_NULLPTR(input); + + // Check start dimensions for being non-negative + ARM_COMPUTE_RETURN_ERROR_ON(std::any_of(starts.cbegin(), starts.cbegin() + starts.num_dimensions(), [](int i) + { + return i < 0; + })); + + // Get absolute end coordinates + const Coordinates ends_abs = arm_compute::helpers::tensor_transform::slice_absolute_end_coords(input->tensor_shape(), ends); + + return CLStridedSliceKernel::validate(input, output, starts, ends_abs, BiStrides(), 0, 0, 0); +} +} // namespace arm_compute diff --git a/tests/datasets/SliceOperationsDataset.h b/tests/datasets/SliceOperationsDataset.h new file mode 100644 index 0000000000..b6df4040fd --- /dev/null +++ b/tests/datasets/SliceOperationsDataset.h @@ -0,0 +1,286 @@ +/* + * Copyright (c) 2018 ARM Limited. + * + * SPDX-License-Identifier: MIT + * + * Permission is hereby granted, free of charge, to any person obtaining a copy + * of this software and associated documentation files (the "Software"), to + * deal in the Software without restriction, including without limitation the + * rights to use, copy, modify, merge, publish, distribute, sublicense, and/or + * sell copies of the Software, and to permit persons to whom the Software is + * furnished to do so, subject to the following conditions: + * + * The above copyright notice and this permission notice shall be included in all + * copies or substantial portions of the Software. + * + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE + * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE + * SOFTWARE. + */ +#ifndef ARM_COMPUTE_TEST_STRIDED_SLICE_DATASET +#define ARM_COMPUTE_TEST_STRIDED_SLICE_DATASET + +#include "utils/TypePrinter.h" + +#include "arm_compute/core/Types.h" + +namespace arm_compute +{ +namespace test +{ +namespace datasets +{ +class SliceDataset +{ +public: + using type = std::tuple; + + struct iterator + { + iterator(std::vector::const_iterator tensor_shapes_it, + std::vector::const_iterator starts_values_it, + std::vector::const_iterator ends_values_it) + : _tensor_shapes_it{ std::move(tensor_shapes_it) }, + _starts_values_it{ std::move(starts_values_it) }, + _ends_values_it{ std::move(ends_values_it) } + { + } + + std::string description() const + { + std::stringstream description; + description << "Shape=" << *_tensor_shapes_it << ":"; + description << "Starts=" << *_starts_values_it << ":"; + description << "Ends=" << *_ends_values_it << ":"; + return description.str(); + } + + SliceDataset::type operator*() const + { + return std::make_tuple(*_tensor_shapes_it, *_starts_values_it, *_ends_values_it); + } + + iterator &operator++() + { + ++_tensor_shapes_it; + ++_starts_values_it; + ++_ends_values_it; + return *this; + } + + private: + std::vector::const_iterator _tensor_shapes_it; + std::vector::const_iterator _starts_values_it; + std::vector::const_iterator _ends_values_it; + }; + + iterator begin() const + { + return iterator(_tensor_shapes.begin(), _starts_values.begin(), _ends_values.begin()); + } + + int size() const + { + return std::min(_tensor_shapes.size(), std::min(_starts_values.size(), _ends_values.size())); + } + + void add_config(TensorShape shape, Coordinates starts, Coordinates ends) + { + _tensor_shapes.emplace_back(std::move(shape)); + _starts_values.emplace_back(std::move(starts)); + _ends_values.emplace_back(std::move(ends)); + } + +protected: + SliceDataset() = default; + SliceDataset(SliceDataset &&) = default; + +private: + std::vector _tensor_shapes{}; + std::vector _starts_values{}; + std::vector _ends_values{}; +}; + +class StridedSliceDataset +{ +public: + using type = std::tuple; + + struct iterator + { + iterator(std::vector::const_iterator tensor_shapes_it, + std::vector::const_iterator starts_values_it, + std::vector::const_iterator ends_values_it, + std::vector::const_iterator strides_values_it, + std::vector::const_iterator begin_mask_values_it, + std::vector::const_iterator end_mask_values_it, + std::vector::const_iterator shrink_mask_values_it) + : _tensor_shapes_it{ std::move(tensor_shapes_it) }, + _starts_values_it{ std::move(starts_values_it) }, + _ends_values_it{ std::move(ends_values_it) }, + _strides_values_it{ std::move(strides_values_it) }, + _begin_mask_values_it{ std::move(begin_mask_values_it) }, + _end_mask_values_it{ std::move(end_mask_values_it) }, + _shrink_mask_values_it{ std::move(shrink_mask_values_it) } + { + } + + std::string description() const + { + std::stringstream description; + description << "Shape=" << *_tensor_shapes_it << ":"; + description << "Starts=" << *_starts_values_it << ":"; + description << "Ends=" << *_ends_values_it << ":"; + description << "Strides=" << *_strides_values_it << ":"; + description << "BeginMask=" << *_begin_mask_values_it << ":"; + description << "EndMask=" << *_end_mask_values_it << ":"; + description << "ShrinkMask=" << *_shrink_mask_values_it << ":"; + return description.str(); + } + + StridedSliceDataset::type operator*() const + { + return std::make_tuple(*_tensor_shapes_it, + *_starts_values_it, *_ends_values_it, *_strides_values_it, + *_begin_mask_values_it, *_end_mask_values_it, *_shrink_mask_values_it); + } + + iterator &operator++() + { + ++_tensor_shapes_it; + ++_starts_values_it; + ++_ends_values_it; + ++_strides_values_it; + ++_begin_mask_values_it; + ++_end_mask_values_it; + ++_shrink_mask_values_it; + + return *this; + } + + private: + std::vector::const_iterator _tensor_shapes_it; + std::vector::const_iterator _starts_values_it; + std::vector::const_iterator _ends_values_it; + std::vector::const_iterator _strides_values_it; + std::vector::const_iterator _begin_mask_values_it; + std::vector::const_iterator _end_mask_values_it; + std::vector::const_iterator _shrink_mask_values_it; + }; + + iterator begin() const + { + return iterator(_tensor_shapes.begin(), + _starts_values.begin(), _ends_values.begin(), _strides_values.begin(), + _begin_mask_values.begin(), _end_mask_values.begin(), _shrink_mask_values.begin()); + } + + int size() const + { + return std::min(_tensor_shapes.size(), std::min(_starts_values.size(), std::min(_ends_values.size(), _strides_values.size()))); + } + + void add_config(TensorShape shape, + Coordinates starts, Coordinates ends, BiStrides strides, + int32_t begin_mask = 0, int32_t end_mask = 0, int32_t shrink_mask = 0) + { + _tensor_shapes.emplace_back(std::move(shape)); + _starts_values.emplace_back(std::move(starts)); + _ends_values.emplace_back(std::move(ends)); + _strides_values.emplace_back(std::move(strides)); + _begin_mask_values.emplace_back(std::move(begin_mask)); + _end_mask_values.emplace_back(std::move(end_mask)); + _shrink_mask_values.emplace_back(std::move(shrink_mask)); + } + +protected: + StridedSliceDataset() = default; + StridedSliceDataset(StridedSliceDataset &&) = default; + +private: + std::vector _tensor_shapes{}; + std::vector _starts_values{}; + std::vector _ends_values{}; + std::vector _strides_values{}; + std::vector _begin_mask_values{}; + std::vector _end_mask_values{}; + std::vector _shrink_mask_values{}; +}; + +class SmallSliceDataset final : public SliceDataset +{ +public: + SmallSliceDataset() + { + // 1D + add_config(TensorShape(15U), Coordinates(4), Coordinates(9)); + add_config(TensorShape(15U), Coordinates(0), Coordinates(-1)); + // 2D + add_config(TensorShape(15U, 16U), Coordinates(0, 1), Coordinates(5, -1)); + add_config(TensorShape(15U, 16U), Coordinates(4, 1), Coordinates(12, -1)); + // 3D + add_config(TensorShape(15U, 16U, 4U), Coordinates(0, 1, 2), Coordinates(5, -1, 4)); + add_config(TensorShape(15U, 16U, 4U), Coordinates(0, 1, 2), Coordinates(5, -1, 4)); + // 4D + add_config(TensorShape(15U, 16U, 4U, 12U), Coordinates(0, 1, 2, 2), Coordinates(5, -1, 4, 5)); + } +}; + +class LargeSliceDataset final : public SliceDataset +{ +public: + LargeSliceDataset() + { + // 1D + add_config(TensorShape(1025U), Coordinates(128), Coordinates(-100)); + // 2D + add_config(TensorShape(372U, 68U), Coordinates(128, 7), Coordinates(368, -1)); + // 3D + add_config(TensorShape(372U, 68U, 12U), Coordinates(128, 7, 2), Coordinates(368, -1, 4)); + // 4D + add_config(TensorShape(372U, 68U, 7U, 4U), Coordinates(128, 7, 2), Coordinates(368, 17, 5)); + } +}; + +class SmallStridedSliceDataset final : public StridedSliceDataset +{ +public: + SmallStridedSliceDataset() + { + // 1D + add_config(TensorShape(15U), Coordinates(0), Coordinates(5), BiStrides(2)); + add_config(TensorShape(15U), Coordinates(-1), Coordinates(-8), BiStrides(-2)); + // 2D + add_config(TensorShape(15U, 16U), Coordinates(0, 1), Coordinates(5, -1), BiStrides(2, 1)); + add_config(TensorShape(15U, 16U), Coordinates(4, 1), Coordinates(12, -1), BiStrides(2, 1), 1); + // 3D + add_config(TensorShape(15U, 16U, 4U), Coordinates(0, 1, 2), Coordinates(5, -1, 4), BiStrides(2, 1, 2)); + add_config(TensorShape(15U, 16U, 4U), Coordinates(0, 1, 2), Coordinates(5, -1, 4), BiStrides(2, 1, 2), 0, 1); + // 4D + add_config(TensorShape(15U, 16U, 4U, 12U), Coordinates(0, 1, 2, 2), Coordinates(5, -1, 4, 5), BiStrides(2, 1, 2, 3)); + } +}; + +class LargeStridedSliceDataset final : public StridedSliceDataset +{ +public: + LargeStridedSliceDataset() + { + // 1D + add_config(TensorShape(1025U), Coordinates(128), Coordinates(-100), BiStrides(20)); + // 2D + add_config(TensorShape(372U, 68U), Coordinates(128, 7), Coordinates(368, -30), BiStrides(10, 7)); + // 3D + add_config(TensorShape(372U, 68U, 12U), Coordinates(128, 7, -1), Coordinates(368, -30, -5), BiStrides(14, 7, -2)); + // 4D + add_config(TensorShape(372U, 68U, 7U, 4U), Coordinates(128, 7, 2), Coordinates(368, -30, 5), BiStrides(20, 7, 2), 1, 1); + } +}; +} // namespace datasets +} // namespace test +} // namespace arm_compute +#endif /* ARM_COMPUTE_TEST_STRIDED_SLICE_DATASET */ diff --git a/tests/datasets/StridedSliceDataset.h b/tests/datasets/StridedSliceDataset.h deleted file mode 100644 index 00f19920b8..0000000000 --- a/tests/datasets/StridedSliceDataset.h +++ /dev/null @@ -1,181 +0,0 @@ -/* - * Copyright (c) 2018 ARM Limited. - * - * SPDX-License-Identifier: MIT - * - * Permission is hereby granted, free of charge, to any person obtaining a copy - * of this software and associated documentation files (the "Software"), to - * deal in the Software without restriction, including without limitation the - * rights to use, copy, modify, merge, publish, distribute, sublicense, and/or - * sell copies of the Software, and to permit persons to whom the Software is - * furnished to do so, subject to the following conditions: - * - * The above copyright notice and this permission notice shall be included in all - * copies or substantial portions of the Software. - * - * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR - * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, - * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE - * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER - * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, - * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE - * SOFTWARE. - */ -#ifndef ARM_COMPUTE_TEST_STRIDED_SLICE_DATASET -#define ARM_COMPUTE_TEST_STRIDED_SLICE_DATASET - -#include "utils/TypePrinter.h" - -#include "arm_compute/core/Types.h" - -namespace arm_compute -{ -namespace test -{ -namespace datasets -{ -class StridedSliceDataset -{ -public: - using type = std::tuple; - - struct iterator - { - iterator(std::vector::const_iterator tensor_shapes_it, - std::vector::const_iterator starts_values_it, - std::vector::const_iterator ends_values_it, - std::vector::const_iterator strides_values_it, - std::vector::const_iterator begin_mask_values_it, - std::vector::const_iterator end_mask_values_it, - std::vector::const_iterator shrink_mask_values_it) - : _tensor_shapes_it{ std::move(tensor_shapes_it) }, - _starts_values_it{ std::move(starts_values_it) }, - _ends_values_it{ std::move(ends_values_it) }, - _strides_values_it{ std::move(strides_values_it) }, - _begin_mask_values_it{ std::move(begin_mask_values_it) }, - _end_mask_values_it{ std::move(end_mask_values_it) }, - _shrink_mask_values_it{ std::move(shrink_mask_values_it) } - { - } - - std::string description() const - { - std::stringstream description; - description << "Shape=" << *_tensor_shapes_it << ":"; - description << "Starts=" << *_starts_values_it << ":"; - description << "Ends=" << *_ends_values_it << ":"; - description << "Strides=" << *_strides_values_it << ":"; - description << "BeginMask=" << *_begin_mask_values_it << ":"; - description << "EndMask=" << *_end_mask_values_it << ":"; - description << "ShrinkMask=" << *_shrink_mask_values_it << ":"; - return description.str(); - } - - StridedSliceDataset::type operator*() const - { - return std::make_tuple(*_tensor_shapes_it, - *_starts_values_it, *_ends_values_it, *_strides_values_it, - *_begin_mask_values_it, *_end_mask_values_it, *_shrink_mask_values_it); - } - - iterator &operator++() - { - ++_tensor_shapes_it; - ++_starts_values_it; - ++_ends_values_it; - ++_strides_values_it; - ++_begin_mask_values_it; - ++_end_mask_values_it; - ++_shrink_mask_values_it; - - return *this; - } - - private: - std::vector::const_iterator _tensor_shapes_it; - std::vector::const_iterator _starts_values_it; - std::vector::const_iterator _ends_values_it; - std::vector::const_iterator _strides_values_it; - std::vector::const_iterator _begin_mask_values_it; - std::vector::const_iterator _end_mask_values_it; - std::vector::const_iterator _shrink_mask_values_it; - }; - - iterator begin() const - { - return iterator(_tensor_shapes.begin(), - _starts_values.begin(), _ends_values.begin(), _strides_values.begin(), - _begin_mask_values.begin(), _end_mask_values.begin(), _shrink_mask_values.begin()); - } - - int size() const - { - return std::min(_tensor_shapes.size(), std::min(_starts_values.size(), std::min(_ends_values.size(), _strides_values.size()))); - } - - void add_config(TensorShape shape, - Coordinates starts, Coordinates ends, BiStrides strides, - int32_t begin_mask = 0, int32_t end_mask = 0, int32_t shrink_mask = 0) - { - _tensor_shapes.emplace_back(std::move(shape)); - _starts_values.emplace_back(std::move(starts)); - _ends_values.emplace_back(std::move(ends)); - _strides_values.emplace_back(std::move(strides)); - _begin_mask_values.emplace_back(std::move(begin_mask)); - _end_mask_values.emplace_back(std::move(end_mask)); - _shrink_mask_values.emplace_back(std::move(shrink_mask)); - } - -protected: - StridedSliceDataset() = default; - StridedSliceDataset(StridedSliceDataset &&) = default; - -private: - std::vector _tensor_shapes{}; - std::vector _starts_values{}; - std::vector _ends_values{}; - std::vector _strides_values{}; - std::vector _begin_mask_values{}; - std::vector _end_mask_values{}; - std::vector _shrink_mask_values{}; -}; - -class SmallStridedSliceDataset final : public StridedSliceDataset -{ -public: - SmallStridedSliceDataset() - { - // 1D - add_config(TensorShape(15U), Coordinates(0), Coordinates(5), BiStrides(2)); - add_config(TensorShape(15U), Coordinates(-1), Coordinates(-8), BiStrides(-2)); - // 2D - add_config(TensorShape(15U, 16U), Coordinates(0, 1), Coordinates(5, -1), BiStrides(2, 1)); - add_config(TensorShape(15U, 16U), Coordinates(4, 1), Coordinates(12, -1), BiStrides(2, 1), 1); - // 3D - add_config(TensorShape(15U, 16U, 4U), Coordinates(0, 1, 2), Coordinates(5, -1, 4), BiStrides(2, 1, 2)); - add_config(TensorShape(15U, 16U, 4U), Coordinates(0, 1, 2), Coordinates(5, -1, 4), BiStrides(2, 1, 2), 0, 1); - // 4D - add_config(TensorShape(15U, 16U, 4U, 12U), Coordinates(0, 1, 2, 2), Coordinates(5, -1, 4, 5), BiStrides(2, 1, 2, 3)); - } -}; - -class LargeStridedSliceDataset final : public StridedSliceDataset -{ -public: - LargeStridedSliceDataset() - { - // 1D - add_config(TensorShape(1025U), Coordinates(128), Coordinates(-100), BiStrides(20)); - // 2D - add_config(TensorShape(372U, 68U), Coordinates(128U, 7U), Coordinates(368U, -30), BiStrides(10, 7)); - // 3D - add_config(TensorShape(372U, 68U, 12U), Coordinates(128U, 7U, -1), Coordinates(368U, -30, -5), BiStrides(14, 7, -2)); - // 4D - add_config(TensorShape(372U, 68U, 7U, 4U), Coordinates(128U, 7U, 2U), Coordinates(368U, -30, 5U), BiStrides(20, 7, 2), 1, 1); - } -}; - -} // namespace datasets -} // namespace test -} // namespace arm_compute -#endif /* ARM_COMPUTE_TEST_STRIDED_SLICE_DATASET */ diff --git a/tests/validation/CL/Slice.cpp b/tests/validation/CL/Slice.cpp new file mode 100644 index 0000000000..91c2e90c07 --- /dev/null +++ b/tests/validation/CL/Slice.cpp @@ -0,0 +1,133 @@ +/* + * Copyright (c) 2018 ARM Limited. + * + * SPDX-License-Identifier: MIT + * + * Permission is hereby granted, free of charge, to any person obtaining a copy + * of this software and associated documentation files (the "Software"), to + * deal in the Software without restriction, including without limitation the + * rights to use, copy, modify, merge, publish, distribute, sublicense, and/or + * sell copies of the Software, and to permit persons to whom the Software is + * furnished to do so, subject to the following conditions: + * + * The above copyright notice and this permission notice shall be included in all + * copies or substantial portions of the Software. + * + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE + * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE + * SOFTWARE. + */ +#include "arm_compute/core/Types.h" +#include "arm_compute/runtime/CL/CLTensor.h" +#include "arm_compute/runtime/CL/CLTensorAllocator.h" +#include "arm_compute/runtime/CL/functions/CLSlice.h" + +#include "tests/CL/CLAccessor.h" +#include "tests/datasets/SliceOperationsDataset.h" +#include "tests/framework/Asserts.h" +#include "tests/framework/Macros.h" +#include "tests/framework/datasets/Datasets.h" +#include "tests/validation/Validation.h" +#include "tests/validation/fixtures/SliceOperationsFixtures.h" + +namespace arm_compute +{ +namespace test +{ +namespace validation +{ +TEST_SUITE(CL) +TEST_SUITE(Slice) + +// *INDENT-OFF* +// clang-format off +DATA_TEST_CASE(Validate, framework::DatasetMode::ALL, zip(zip(zip( + framework::dataset::make("InputInfo", { TensorInfo(TensorShape(27U, 3U, 2U, 5U, 3U), 1, DataType::F32), // Invalid input shape + TensorInfo(TensorShape(27U, 3U, 2U), 1, DataType::F32), // Negative begin + TensorInfo(TensorShape(27U, 3U, 2U), 1, DataType::F32), // Big number of coordinates + TensorInfo(TensorShape(27U, 3U, 2U), 1, DataType::F32) + }), + framework::dataset::make("Starts", { Coordinates(3, 1, 0), Coordinates(-3, 1, 0), Coordinates(3, 1, 0), Coordinates(3, 1, 0) })), + framework::dataset::make("Ends", { Coordinates(13, 3, 0), Coordinates(13, 3, 1), Coordinates(13, 3, 1, 1), Coordinates(13, 3, 1) })), + framework::dataset::make("Expected", { false, false, false, true })), + input_info, starts, ends, expected) +{ + TensorInfo output_info; + const Status status = CLSlice::validate(&input_info.clone()->set_is_resizable(false), &output_info, starts, ends); + ARM_COMPUTE_EXPECT(bool(status) == expected, framework::LogLevel::ERRORS); +} +// clang-format on +// *INDENT-ON* + +DATA_TEST_CASE(Configuration, + framework::DatasetMode::ALL, + combine(arm_compute::test::datasets::SmallSliceDataset(), framework::dataset::make("DataType", { DataType::F16, DataType::F32 })), + shape, starts, ends, data_type) +{ + // Create tensors + CLTensor src = create_tensor(shape, data_type); + CLTensor dst; + + // Create and Configure function + CLSlice slice; + slice.configure(&src, &dst, starts, ends); + + // Validate valid region + const ValidRegion valid_region = shape_to_valid_region(dst.info()->tensor_shape()); + validate(dst.info()->valid_region(), valid_region); +} + +template +using CLSliceFixture = SliceFixture; + +TEST_SUITE(Float) +TEST_SUITE(FP16) +FIXTURE_DATA_TEST_CASE(RunSmall, + CLSliceFixture, + framework::DatasetMode::PRECOMMIT, + combine(datasets::SmallSliceDataset(), framework::dataset::make("DataType", DataType::F16))) +{ + // Validate output + validate(CLAccessor(_target), _reference); +} + +FIXTURE_DATA_TEST_CASE(RunLarge, + CLSliceFixture, + framework::DatasetMode::NIGHTLY, + combine(datasets::LargeSliceDataset(), framework::dataset::make("DataType", DataType::F16))) +{ + // Validate output + validate(CLAccessor(_target), _reference); +} +TEST_SUITE_END() // FP16 + +TEST_SUITE(FP32) +FIXTURE_DATA_TEST_CASE(RunSmall, + CLSliceFixture, + framework::DatasetMode::PRECOMMIT, + combine(datasets::SmallSliceDataset(), framework::dataset::make("DataType", DataType::F32))) +{ + // Validate output + validate(CLAccessor(_target), _reference); +} + +FIXTURE_DATA_TEST_CASE(RunLarge, + CLSliceFixture, + framework::DatasetMode::NIGHTLY, + combine(datasets::LargeSliceDataset(), framework::dataset::make("DataType", DataType::F32))) +{ + // Validate output + validate(CLAccessor(_target), _reference); +} +TEST_SUITE_END() // FP32 +TEST_SUITE_END() // Float + +TEST_SUITE_END() // Slice +TEST_SUITE_END() // CL +} // namespace validation +} // namespace test +} // namespace arm_compute diff --git a/tests/validation/CL/StridedSlice.cpp b/tests/validation/CL/StridedSlice.cpp index 824f7b1cf3..00eee168f9 100644 --- a/tests/validation/CL/StridedSlice.cpp +++ b/tests/validation/CL/StridedSlice.cpp @@ -28,12 +28,12 @@ #include "tests/CL/CLAccessor.h" #include "tests/PaddingCalculator.h" -#include "tests/datasets/StridedSliceDataset.h" +#include "tests/datasets/SliceOperationsDataset.h" #include "tests/framework/Asserts.h" #include "tests/framework/Macros.h" #include "tests/framework/datasets/Datasets.h" #include "tests/validation/Validation.h" -#include "tests/validation/fixtures/StridedSliceFixture.h" +#include "tests/validation/fixtures/SliceOperationsFixtures.h" namespace arm_compute { diff --git a/tests/validation/fixtures/SliceOperationsFixtures.h b/tests/validation/fixtures/SliceOperationsFixtures.h new file mode 100644 index 0000000000..018bafba6c --- /dev/null +++ b/tests/validation/fixtures/SliceOperationsFixtures.h @@ -0,0 +1,182 @@ +/* + * Copyright (c) 2018 ARM Limited. + * + * SPDX-License-Identifier: MIT + * + * Permission is hereby granted, free of charge, to any person obtaining a copy + * of this software and associated documentation files (the "Software"), to + * deal in the Software without restriction, including without limitation the + * rights to use, copy, modify, merge, publish, distribute, sublicense, and/or + * sell copies of the Software, and to permit persons to whom the Software is + * furnished to do so, subject to the following conditions: + * + * The above copyright notice and this permission notice shall be included in all + * copies or substantial portions of the Software. + * + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE + * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE + * SOFTWARE. + */ +#ifndef ARM_COMPUTE_TEST_STRIDED_SLICE_FIXTURE +#define ARM_COMPUTE_TEST_STRIDED_SLICE_FIXTURE + +#include "arm_compute/core/TensorShape.h" +#include "arm_compute/core/Types.h" + +#include "tests/AssetsLibrary.h" +#include "tests/Globals.h" +#include "tests/IAccessor.h" +#include "tests/RawLutAccessor.h" +#include "tests/framework/Asserts.h" +#include "tests/framework/Fixture.h" +#include "tests/validation/Helpers.h" +#include "tests/validation/reference/SliceOperations.h" + +namespace arm_compute +{ +namespace test +{ +namespace validation +{ +template +class SliceFixture : public framework::Fixture +{ +public: + template + void setup(TensorShape shape, Coordinates starts, Coordinates ends, DataType data_type) + { + _target = compute_target(shape, starts, ends, data_type); + _reference = compute_reference(shape, starts, ends, data_type); + } + +protected: + template + void fill(U &&tensor, int i) + { + library->fill_tensor_uniform(tensor, i); + } + + TensorType compute_target(const TensorShape &shape, const Coordinates &starts, const Coordinates &ends, DataType data_type) + { + // Create tensors + TensorType src = create_tensor(shape, data_type); + TensorType dst; + + // Create and configure function + FunctionType slice; + slice.configure(&src, &dst, starts, ends); + + ARM_COMPUTE_EXPECT(src.info()->is_resizable(), framework::LogLevel::ERRORS); + ARM_COMPUTE_EXPECT(dst.info()->is_resizable(), framework::LogLevel::ERRORS); + + // Allocate tensors + src.allocator()->allocate(); + dst.allocator()->allocate(); + + ARM_COMPUTE_EXPECT(!src.info()->is_resizable(), framework::LogLevel::ERRORS); + ARM_COMPUTE_EXPECT(!dst.info()->is_resizable(), framework::LogLevel::ERRORS); + + // Fill tensors + fill(AccessorType(src), 0); + fill(AccessorType(dst), 1); + + // Compute function + slice.run(); + + return dst; + } + + SimpleTensor compute_reference(const TensorShape &shape, const Coordinates &starts, const Coordinates &ends, DataType data_type) + { + // Create reference + SimpleTensor src{ shape, data_type }; + + // Fill reference + fill(src, 0); + + return reference::slice(src, starts, ends); + } + + TensorType _target{}; + SimpleTensor _reference{}; +}; + +template +class StridedSliceFixture : public framework::Fixture +{ +public: + template + void setup(TensorShape shape, + Coordinates starts, Coordinates ends, BiStrides strides, + int32_t begin_mask, int32_t end_mask, int32_t shrink_mask, + DataType data_type) + { + _target = compute_target(shape, starts, ends, strides, begin_mask, end_mask, shrink_mask, data_type); + _reference = compute_reference(shape, starts, ends, strides, begin_mask, end_mask, shrink_mask, data_type); + } + +protected: + template + void fill(U &&tensor, int i) + { + library->fill_tensor_uniform(tensor, i); + } + + TensorType compute_target(const TensorShape &shape, + const Coordinates &starts, const Coordinates &ends, const BiStrides &strides, + int32_t begin_mask, int32_t end_mask, int32_t shrink_mask, + DataType data_type) + { + // Create tensors + TensorType src = create_tensor(shape, data_type); + TensorType dst; + + // Create and configure function + FunctionType strided_slice; + strided_slice.configure(&src, &dst, starts, ends, strides, begin_mask, end_mask, shrink_mask); + + ARM_COMPUTE_EXPECT(src.info()->is_resizable(), framework::LogLevel::ERRORS); + ARM_COMPUTE_EXPECT(dst.info()->is_resizable(), framework::LogLevel::ERRORS); + + // Allocate tensors + src.allocator()->allocate(); + dst.allocator()->allocate(); + + ARM_COMPUTE_EXPECT(!src.info()->is_resizable(), framework::LogLevel::ERRORS); + ARM_COMPUTE_EXPECT(!dst.info()->is_resizable(), framework::LogLevel::ERRORS); + + // Fill tensors + fill(AccessorType(src), 0); + fill(AccessorType(dst), 1); + + // Compute function + strided_slice.run(); + + return dst; + } + + SimpleTensor compute_reference(const TensorShape &shape, + const Coordinates &starts, const Coordinates &ends, const BiStrides &strides, + int32_t begin_mask, int32_t end_mask, int32_t shrink_mask, + DataType data_type) + { + // Create reference + SimpleTensor src{ shape, data_type }; + + // Fill reference + fill(src, 0); + + return reference::strided_slice(src, starts, ends, strides, begin_mask, end_mask, shrink_mask); + } + + TensorType _target{}; + SimpleTensor _reference{}; +}; +} // namespace validation +} // namespace test +} // namespace arm_compute +#endif /* ARM_COMPUTE_TEST_STRIDED_SLICE_FIXTURE */ diff --git a/tests/validation/fixtures/StridedSliceFixture.h b/tests/validation/fixtures/StridedSliceFixture.h deleted file mode 100644 index 26442ad834..0000000000 --- a/tests/validation/fixtures/StridedSliceFixture.h +++ /dev/null @@ -1,119 +0,0 @@ -/* - * Copyright (c) 2018 ARM Limited. - * - * SPDX-License-Identifier: MIT - * - * Permission is hereby granted, free of charge, to any person obtaining a copy - * of this software and associated documentation files (the "Software"), to - * deal in the Software without restriction, including without limitation the - * rights to use, copy, modify, merge, publish, distribute, sublicense, and/or - * sell copies of the Software, and to permit persons to whom the Software is - * furnished to do so, subject to the following conditions: - * - * The above copyright notice and this permission notice shall be included in all - * copies or substantial portions of the Software. - * - * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR - * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, - * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE - * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER - * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, - * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE - * SOFTWARE. - */ -#ifndef ARM_COMPUTE_TEST_STRIDED_SLICE_FIXTURE -#define ARM_COMPUTE_TEST_STRIDED_SLICE_FIXTURE - -#include "arm_compute/core/TensorShape.h" -#include "arm_compute/core/Types.h" - -#include "tests/AssetsLibrary.h" -#include "tests/Globals.h" -#include "tests/IAccessor.h" -#include "tests/RawLutAccessor.h" -#include "tests/framework/Asserts.h" -#include "tests/framework/Fixture.h" -#include "tests/validation/Helpers.h" -#include "tests/validation/reference/StridedSlice.h" - -namespace arm_compute -{ -namespace test -{ -namespace validation -{ -template -class StridedSliceFixture : public framework::Fixture -{ -public: - template - void setup(TensorShape shape, - Coordinates starts, Coordinates ends, BiStrides strides, - int32_t begin_mask, int32_t end_mask, int32_t shrink_mask, - DataType data_type) - { - _target = compute_target(shape, starts, ends, strides, begin_mask, end_mask, shrink_mask, data_type); - _reference = compute_reference(shape, starts, ends, strides, begin_mask, end_mask, shrink_mask, data_type); - } - -protected: - template - void fill(U &&tensor, int i) - { - library->fill_tensor_uniform(tensor, i); - } - - TensorType compute_target(const TensorShape &shape, - const Coordinates &starts, const Coordinates &ends, const BiStrides &strides, - int32_t begin_mask, int32_t end_mask, int32_t shrink_mask, - DataType data_type) - { - // Create tensors - TensorType src = create_tensor(shape, data_type); - TensorType dst; - - // Create and configure function - FunctionType strided_slice; - strided_slice.configure(&src, &dst, starts, ends, strides, begin_mask, end_mask, shrink_mask); - - ARM_COMPUTE_EXPECT(src.info()->is_resizable(), framework::LogLevel::ERRORS); - ARM_COMPUTE_EXPECT(dst.info()->is_resizable(), framework::LogLevel::ERRORS); - - // Allocate tensors - src.allocator()->allocate(); - dst.allocator()->allocate(); - - ARM_COMPUTE_EXPECT(!src.info()->is_resizable(), framework::LogLevel::ERRORS); - ARM_COMPUTE_EXPECT(!dst.info()->is_resizable(), framework::LogLevel::ERRORS); - - // Fill tensors - fill(AccessorType(src), 0); - fill(AccessorType(dst), 1); - - // Compute function - strided_slice.run(); - - return dst; - } - - SimpleTensor compute_reference(const TensorShape &shape, - const Coordinates &starts, const Coordinates &ends, const BiStrides &strides, - int32_t begin_mask, int32_t end_mask, int32_t shrink_mask, - DataType data_type) - { - // Create reference - SimpleTensor src{ shape, data_type }; - - // Fill reference - fill(src, 0); - - return reference::strided_slice(src, starts, ends, strides, begin_mask, end_mask, shrink_mask); - } - - TensorType _target{}; - SimpleTensor _reference{}; -}; -} // namespace validation -} // namespace test -} // namespace arm_compute -#endif /* ARM_COMPUTE_TEST_STRIDED_SLICE_FIXTURE */ diff --git a/tests/validation/reference/SliceOperations.cpp b/tests/validation/reference/SliceOperations.cpp new file mode 100644 index 0000000000..04b5b98453 --- /dev/null +++ b/tests/validation/reference/SliceOperations.cpp @@ -0,0 +1,137 @@ +/* + * Copyright (c) 2018 ARM Limited. + * + * SPDX-License-Identifier: MIT + * + * Permission is hereby granted, free of charge, to any person obtaining a copy + * of this software and associated documentation files (the "Software"), to + * deal in the Software without restriction, including without limitation the + * rights to use, copy, modify, merge, publish, distribute, sublicense, and/or + * sell copies of the Software, and to permit persons to whom the Software is + * furnished to do so, subject to the following conditions: + * + * The above copyright notice and this permission notice shall be included in all + * copies or substantial portions of the Software. + * + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE + * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE + * SOFTWARE. + */ +#include "SliceOperations.h" + +#include "arm_compute/core/utils/helpers/tensor_transform.h" + +namespace arm_compute +{ +namespace test +{ +namespace validation +{ +namespace reference +{ +template +SimpleTensor slice(const SimpleTensor &src, Coordinates starts, Coordinates ends) +{ + using namespace arm_compute::helpers::tensor_transform; + + // Validation checks + ARM_COMPUTE_ERROR_ON(src.shape().num_dimensions() > 4); + ARM_COMPUTE_ERROR_ON(starts.num_dimensions() > src.shape().num_dimensions()); + ARM_COMPUTE_ERROR_ON(std::any_of(starts.cbegin(), starts.cbegin() + starts.num_dimensions(), [](int i) + { + return i < 0; + })); + ARM_COMPUTE_ERROR_ON(ends.num_dimensions() > src.shape().num_dimensions()); + + // Get source shape + const TensorShape &src_shape = src.shape(); + + // Get actual end + Coordinates ends_abs = slice_absolute_end_coords(src_shape, ends); + + // Get destination shape + TensorShape dst_shape = compute_slice_output_shape(src_shape, starts, ends_abs); + + // Create destination tensor + SimpleTensor dst{ dst_shape, src.data_type(), 1 }; + + // Perform slice + Window win; + win.use_tensor_dimensions(dst_shape); + execute_window_loop(win, [&](const Coordinates & id) + { + Coordinates offset; + for(unsigned int i = 0; i < id.num_dimensions(); ++i) + { + offset.set(i, starts[i] + id[i]); + } + *reinterpret_cast(dst(id)) = *reinterpret_cast(src(offset)); + }); + + return dst; +} + +template SimpleTensor slice(const SimpleTensor &src, Coordinates starts, Coordinates ends); +template SimpleTensor slice(const SimpleTensor &src, Coordinates starts, Coordinates ends); + +template +SimpleTensor strided_slice(const SimpleTensor &src, + Coordinates starts, Coordinates ends, BiStrides strides, + int32_t begin_mask, int32_t end_mask, int32_t shrink_axis_mask) +{ + using namespace arm_compute::helpers::tensor_transform; + + // Validation checks + ARM_COMPUTE_ERROR_ON(src.shape().num_dimensions() > 4); + ARM_COMPUTE_ERROR_ON(starts.num_dimensions() > src.shape().num_dimensions()); + ARM_COMPUTE_ERROR_ON(ends.num_dimensions() > src.shape().num_dimensions()); + ARM_COMPUTE_ERROR_ON(strides.num_dimensions() > src.shape().num_dimensions()); + ARM_COMPUTE_ERROR_ON(std::any_of(strides.cbegin(), strides.cbegin() + strides.num_dimensions(), [](int i) + { + return i == 0; + })); + + // Get source shape + const TensorShape &src_shape = src.shape(); + + // Get actual start, end coordinates and strides + const Coordinates final_strides = strided_slice_strides(src_shape, strides); + const Coordinates starts_abs = strided_slice_absolute_start_coords(src_shape, starts, final_strides, begin_mask); + const Coordinates ends_abs = strided_slice_absolute_end_coords(src_shape, starts_abs, ends, final_strides, end_mask, shrink_axis_mask); + + // Get destination shape + const TensorShape dst_shape = compute_strided_slice_output_shape(src_shape, starts_abs, ends_abs, final_strides); + + // Create destination tensor + SimpleTensor dst{ dst_shape, src.data_type(), 1 }; + + // Perform strided slice + Window win; + win.use_tensor_dimensions(dst_shape); + execute_window_loop(win, [&](const Coordinates & id) + { + Coordinates offset; + for(unsigned int i = 0; i < id.num_dimensions(); ++i) + { + offset.set(i, starts_abs[i] + id[i] * final_strides[i]); + } + *reinterpret_cast(dst(id)) = *reinterpret_cast(src(offset)); + }); + + return dst; +} + +template SimpleTensor strided_slice(const SimpleTensor &src, + Coordinates starts, Coordinates ends, BiStrides strides, + int32_t begin_mask, int32_t end_mask, int32_t shrink_axis_mask); +template SimpleTensor strided_slice(const SimpleTensor &src, + Coordinates starts, Coordinates ends, BiStrides strides, + int32_t begin_mask, int32_t end_mask, int32_t shrink_axis_mask); +} // namespace reference +} // namespace validation +} // namespace test +} // namespace arm_compute diff --git a/tests/validation/reference/SliceOperations.h b/tests/validation/reference/SliceOperations.h new file mode 100644 index 0000000000..89fe2038ed --- /dev/null +++ b/tests/validation/reference/SliceOperations.h @@ -0,0 +1,48 @@ +/* + * Copyright (c) 2018 ARM Limited. + * + * SPDX-License-Identifier: MIT + * + * Permission is hereby granted, free of charge, to any person obtaining a copy + * of this software and associated documentation files (the "Software"), to + * deal in the Software without restriction, including without limitation the + * rights to use, copy, modify, merge, publish, distribute, sublicense, and/or + * sell copies of the Software, and to permit persons to whom the Software is + * furnished to do so, subject to the following conditions: + * + * The above copyright notice and this permission notice shall be included in all + * copies or substantial portions of the Software. + * + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE + * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE + * SOFTWARE. + */ +#ifndef __ARM_COMPUTE_TEST_SLICE_OPERATIONS_H__ +#define __ARM_COMPUTE_TEST_SLICE_OPERATIONS_H__ + +#include "tests/SimpleTensor.h" + +namespace arm_compute +{ +namespace test +{ +namespace validation +{ +namespace reference +{ +template +SimpleTensor slice(const SimpleTensor &src, Coordinates starts, Coordinates ends); + +template +SimpleTensor strided_slice(const SimpleTensor &src, + Coordinates starts, Coordinates ends, BiStrides strides, + int32_t begin_mask, int32_t end_mask, int32_t shrink_axis_mask); +} // namespace reference +} // namespace validation +} // namespace test +} // namespace arm_compute +#endif /* __ARM_COMPUTE_TEST_SLICE_OPERATIONS_H__ */ diff --git a/tests/validation/reference/StridedSlice.cpp b/tests/validation/reference/StridedSlice.cpp deleted file mode 100644 index ac02ce1e71..0000000000 --- a/tests/validation/reference/StridedSlice.cpp +++ /dev/null @@ -1,92 +0,0 @@ -/* - * Copyright (c) 2018 ARM Limited. - * - * SPDX-License-Identifier: MIT - * - * Permission is hereby granted, free of charge, to any person obtaining a copy - * of this software and associated documentation files (the "Software"), to - * deal in the Software without restriction, including without limitation the - * rights to use, copy, modify, merge, publish, distribute, sublicense, and/or - * sell copies of the Software, and to permit persons to whom the Software is - * furnished to do so, subject to the following conditions: - * - * The above copyright notice and this permission notice shall be included in all - * copies or substantial portions of the Software. - * - * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR - * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, - * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE - * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER - * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, - * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE - * SOFTWARE. - */ -#include "StridedSlice.h" - -#include "arm_compute/core/utils/helpers/tensor_transform.h" - -namespace arm_compute -{ -namespace test -{ -namespace validation -{ -namespace reference -{ -template -SimpleTensor strided_slice(const SimpleTensor &src, - Coordinates starts, Coordinates ends, BiStrides strides, - int32_t begin_mask, int32_t end_mask, int32_t shrink_axis_mask) -{ - using namespace arm_compute::helpers::tensor_transform; - - // Validation checks - ARM_COMPUTE_ERROR_ON(src.shape().num_dimensions() > 4); - ARM_COMPUTE_ERROR_ON(starts.num_dimensions() > src.shape().num_dimensions()); - ARM_COMPUTE_ERROR_ON(ends.num_dimensions() > src.shape().num_dimensions()); - ARM_COMPUTE_ERROR_ON(strides.num_dimensions() > src.shape().num_dimensions()); - for(unsigned int i = 0; i < strides.num_dimensions(); ++i) - { - ARM_COMPUTE_ERROR_ON(strides[i] == 0); - } - - // Get source shape - const TensorShape &src_shape = src.shape(); - - // Get actual start, end coordinates and strides - const Coordinates final_strides = strided_slice_strides(src_shape, strides); - const Coordinates starts_abs = strided_slice_absolute_start_coords(src_shape, starts, final_strides, begin_mask); - const Coordinates ends_abs = strided_slice_absolute_end_coords(src_shape, starts_abs, ends, final_strides, end_mask, shrink_axis_mask); - - // Get destination shape - const TensorShape dst_shape = compute_strided_slice_output_shape(src_shape, starts_abs, ends_abs, final_strides); - - // Create destination tensor - SimpleTensor dst{ dst_shape, src.data_type(), 1 }; - - // Perform strided slice - Window win; - win.use_tensor_dimensions(dst_shape); - execute_window_loop(win, [&](const Coordinates & id) - { - Coordinates offset; - for(unsigned int i = 0; i < id.num_dimensions(); ++i) - { - offset.set(i, starts_abs[i] + id[i] * final_strides[i]); - } - *reinterpret_cast(dst(id)) = *reinterpret_cast(src(offset)); - }); - - return dst; -} - -template SimpleTensor strided_slice(const SimpleTensor &src, - Coordinates starts, Coordinates ends, BiStrides strides, - int32_t begin_mask, int32_t end_mask, int32_t shrink_axis_mask); -template SimpleTensor strided_slice(const SimpleTensor &src, - Coordinates starts, Coordinates ends, BiStrides strides, - int32_t begin_mask, int32_t end_mask, int32_t shrink_axis_mask); -} // namespace reference -} // namespace validation -} // namespace test -} // namespace arm_compute diff --git a/tests/validation/reference/StridedSlice.h b/tests/validation/reference/StridedSlice.h deleted file mode 100644 index 805b7e65da..0000000000 --- a/tests/validation/reference/StridedSlice.h +++ /dev/null @@ -1,45 +0,0 @@ -/* - * Copyright (c) 2018 ARM Limited. - * - * SPDX-License-Identifier: MIT - * - * Permission is hereby granted, free of charge, to any person obtaining a copy - * of this software and associated documentation files (the "Software"), to - * deal in the Software without restriction, including without limitation the - * rights to use, copy, modify, merge, publish, distribute, sublicense, and/or - * sell copies of the Software, and to permit persons to whom the Software is - * furnished to do so, subject to the following conditions: - * - * The above copyright notice and this permission notice shall be included in all - * copies or substantial portions of the Software. - * - * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR - * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, - * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE - * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER - * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, - * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE - * SOFTWARE. - */ -#ifndef __ARM_COMPUTE_TEST_STRIDED_SLICE_H__ -#define __ARM_COMPUTE_TEST_STRIDED_SLICE_H__ - -#include "tests/SimpleTensor.h" - -namespace arm_compute -{ -namespace test -{ -namespace validation -{ -namespace reference -{ -template -SimpleTensor strided_slice(const SimpleTensor &src, - Coordinates starts, Coordinates ends, BiStrides strides, - int32_t begin_mask, int32_t end_mask, int32_t shrink_axis_mask); -} // namespace reference -} // namespace validation -} // namespace test -} // namespace arm_compute -#endif /* __ARM_COMPUTE_TEST_STRIDED_SLICE_H__ */ -- cgit v1.2.1