diff options
author | SiCong Li <sicong.li@arm.com> | 2020-11-17 14:09:01 +0000 |
---|---|---|
committer | SiCong Li <sicong.li@arm.com> | 2020-11-17 16:38:52 +0000 |
commit | 04a0706dddc6ca24cb80e3e0789c6b0f54c48b28 (patch) | |
tree | eb136b5fd7bd4f3e9424cf5bcf4504e5a4f158d1 /tests/validation/CL/BatchConcatenateLayer.cpp | |
parent | 1d1bca75f766625140ab0fdf000b6336f013ed83 (diff) | |
download | ComputeLibrary-04a0706dddc6ca24cb80e3e0789c6b0f54c48b28.tar.gz |
COMPMID-3979 Sanitise Padding Removal epic
* Add missing padding immutability asserts in all relevant CL kernels
* Remove unnecessary zero padding validation tests.
Change-Id: If93f9ccbc988e0286f5e7b135f812141476d5da0
Signed-off-by: SiCong Li <sicong.li@arm.com>
Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/4446
Reviewed-by: Gian Marco Iodice <gianmarco.iodice@arm.com>
Comments-Addressed: Arm Jenkins <bsgcomp@arm.com>
Tested-by: Arm Jenkins <bsgcomp@arm.com>
Diffstat (limited to 'tests/validation/CL/BatchConcatenateLayer.cpp')
-rw-r--r-- | tests/validation/CL/BatchConcatenateLayer.cpp | 51 |
1 files changed, 0 insertions, 51 deletions
diff --git a/tests/validation/CL/BatchConcatenateLayer.cpp b/tests/validation/CL/BatchConcatenateLayer.cpp index e5de3a75c7..522a6ab8ee 100644 --- a/tests/validation/CL/BatchConcatenateLayer.cpp +++ b/tests/validation/CL/BatchConcatenateLayer.cpp @@ -39,37 +39,6 @@ namespace test { namespace validation { -namespace -{ -/** Zero padding test */ -bool validate_zero_padding(unsigned int width, unsigned int height, unsigned int channels, unsigned int batches, DataType data_type) -{ - TensorShape src_shape(width, height, channels, batches); - TensorShape dst_shape(width, height, channels, batches * 2); - - // Create tensors - CLTensor src0 = create_tensor<CLTensor>(src_shape, data_type); - CLTensor src1 = create_tensor<CLTensor>(src_shape, data_type); - CLTensor dst = create_tensor<CLTensor>(dst_shape, data_type); - - src0.info()->set_quantization_info(QuantizationInfo(1.f / 256.f, 0)); - src1.info()->set_quantization_info(QuantizationInfo(1.f / 256.f, 0)); - dst.info()->set_quantization_info(QuantizationInfo(1.f / 256.f, 0)); - - ARM_COMPUTE_EXPECT(src0.info()->is_resizable(), framework::LogLevel::ERRORS); - ARM_COMPUTE_EXPECT(src1.info()->is_resizable(), framework::LogLevel::ERRORS); - ARM_COMPUTE_EXPECT(dst.info()->is_resizable(), framework::LogLevel::ERRORS); - - std::vector<const ICLTensor *> srcs = { &src0, &src1 }; - - // Create and configure function - CLConcatenateLayer concat; - concat.configure(srcs, &dst, 3U); - - // Padding can be added along rhs and bias's X dimension - return src0.info()->padding().empty() && src1.info()->padding().empty() && dst.info()->padding().empty(); -} -} TEST_SUITE(CL) TEST_SUITE(BatchConcatenateLayer) @@ -112,26 +81,6 @@ DATA_TEST_CASE(Validate, framework::DatasetMode::ALL, zip(zip(zip( ARM_COMPUTE_EXPECT(is_valid == expected, framework::LogLevel::ERRORS); } -/** Validate zero padding tests - * - * A series of validation tests to check that no padding is added as part of configuration for 4 different scenarios. - * - * Checks performed in order: - * - First dimension multiple of 16 - * - First dimension non-multiple of 16 - * - First dimension less than 16 (vec_size for qasymm8) but multiple - * - First dimension less than 16 (vec_size for qasymm8) non-multiple - * - Tensor with only one element - */ -DATA_TEST_CASE(ValidateZeroPadding, framework::DatasetMode::ALL, zip( -framework::dataset::make("Width", { 32U, 37U, 12U, 13U, 1U }), -framework::dataset::make("DataType", { DataType::F32, DataType::QASYMM8 })), -width, data_type) -{ - const bool one_elem = (width == 1U); - bool status = validate_zero_padding(width, one_elem ? 1U : 17U, one_elem ? 1U : 7U, one_elem ? 1U : 2U, data_type); - ARM_COMPUTE_EXPECT(status, framework::LogLevel::ERRORS); -} // clang-format on // *INDENT-ON* |