From e78ef6fda8f9dafec3a08af917a946ebe18944c5 Mon Sep 17 00:00:00 2001 From: Manuel Bottini Date: Fri, 8 Jan 2021 15:57:11 +0000 Subject: Remove OpenCL padding CLFloorKernel Use of proper vector size with boundary checking loads and stores Resolves: COMPMID-3922 Change-Id: Ib631d499603b860fcfdbe3da903b866a125359a8 Signed-off-by: Manuel Bottini Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/4789 Tested-by: Arm Jenkins Reviewed-by: Michele Di Giorgio Reviewed-by: Gian Marco Iodice Comments-Addressed: Arm Jenkins --- tests/validation/CL/Floor.cpp | 6 ++---- tests/validation/CL/UNIT/CompileContext.cpp | 1 + 2 files changed, 3 insertions(+), 4 deletions(-) (limited to 'tests/validation') diff --git a/tests/validation/CL/Floor.cpp b/tests/validation/CL/Floor.cpp index 2961cfa3f2..a63f16b3c0 100644 --- a/tests/validation/CL/Floor.cpp +++ b/tests/validation/CL/Floor.cpp @@ -1,5 +1,5 @@ /* - * Copyright (c) 2017-2020 Arm Limited. + * Copyright (c) 2017-2021 Arm Limited. * * SPDX-License-Identifier: MIT * @@ -47,18 +47,16 @@ TEST_SUITE(Floor) // clang-format off DATA_TEST_CASE(Validate, framework::DatasetMode::ALL, zip(zip( framework::dataset::make("InputInfo", { TensorInfo(TensorShape(32U, 13U, 2U), 1, DataType::U8), // Wrong data type - TensorInfo(TensorShape(27U, 13U, 2U), 1, DataType::F32), // Window shrink TensorInfo(TensorShape(32U, 13U, 2U), 1, DataType::F32), // Invalid data type combination TensorInfo(TensorShape(32U, 13U, 2U), 1, DataType::F32), // Mismatching shapes TensorInfo(TensorShape(32U, 13U, 2U), 1, DataType::F32), }), framework::dataset::make("OutputInfo",{ TensorInfo(TensorShape(32U, 13U, 2U), 1, DataType::F32), - TensorInfo(TensorShape(27U, 13U, 2U), 1, DataType::F32), TensorInfo(TensorShape(32U, 13U, 2U), 1, DataType::F16), TensorInfo(TensorShape(48U, 11U, 2U), 1, DataType::F32), TensorInfo(TensorShape(32U, 13U, 2U), 1, DataType::F32), })), - framework::dataset::make("Expected", { false, false, false, false, true })), + framework::dataset::make("Expected", { false, false, false, true })), input_info, output_info, expected) { const Status status = CLFloor::validate(&input_info.clone()->set_is_resizable(false), &output_info.clone()->set_is_resizable(false)); diff --git a/tests/validation/CL/UNIT/CompileContext.cpp b/tests/validation/CL/UNIT/CompileContext.cpp index 06f7eae88e..a0541b6a99 100644 --- a/tests/validation/CL/UNIT/CompileContext.cpp +++ b/tests/validation/CL/UNIT/CompileContext.cpp @@ -56,6 +56,7 @@ TEST_CASE(CompileContextCache, framework::DatasetMode::ALL) std::set build_opts; build_opts.emplace("-DDATA_TYPE=float"); build_opts.emplace("-DVEC_SIZE=16"); + build_opts.emplace("-DVEC_SIZE_LEFTOVER=0"); compile_context.create_kernel(kernel_name, program_name, kernel_src.first, kernel_path, build_opts, kernel_src.second); // Check if the program is stored in the cache -- cgit v1.2.1