From 63825e8259508dc7731b6de2e008c5ef8c738d79 Mon Sep 17 00:00:00 2001 From: Giorgio Arena Date: Thu, 25 Mar 2021 14:54:50 +0000 Subject: Implicit padding testing along the X axis on high priority operators Add artificial implicit padding testing for the following fixtures: - Scale - FullyConnected - Pooling - DepthwiseConvolution - DirectConvolution - Winograd - FFT - GEMM/GEMMLowp Create utility function that loops through a list of tensor and adds random padding based on the global seed (only for NHWC layer layout). Remove GEMMLowpAssemblyFixture since it wasn't used Remove some AssetsLibrary headers since they weren't used Resolve COMPMID-4161 Change-Id: Ib6f4f7f113ae69b993d7b2a9e04abbf3de8c99fe Signed-off-by: Giorgio Arena Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/5327 Reviewed-by: Michele Di Giorgio Reviewed-by: Georgios Pinitas Comments-Addressed: Arm Jenkins Tested-by: Arm Jenkins --- tests/validation/fixtures/DepthwiseConvolutionLayerFixture.h | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) (limited to 'tests/validation/fixtures/DepthwiseConvolutionLayerFixture.h') diff --git a/tests/validation/fixtures/DepthwiseConvolutionLayerFixture.h b/tests/validation/fixtures/DepthwiseConvolutionLayerFixture.h index 0aa43d82b4..e87e31f97b 100644 --- a/tests/validation/fixtures/DepthwiseConvolutionLayerFixture.h +++ b/tests/validation/fixtures/DepthwiseConvolutionLayerFixture.h @@ -116,6 +116,9 @@ public: void allocate_and_run_target() { + // TODO: uncomment after COMPMID-4361 + // add_padding_x({ &_src, &_weights, &_biases, &_target }, _data_layout); + // Allocate tensors _src.allocator()->allocate(); _weights.allocator()->allocate(); @@ -131,7 +134,7 @@ public: fill(AccessorType(_src), 0); fill(AccessorType(_weights), 1); fill(AccessorType(_biases), 2); - + if(_mixed_layout) { mix_layout(_dwc, _src, _target); @@ -158,7 +161,6 @@ public: } protected: - void mix_layout(FunctionType &layer, TensorType &src, TensorType &dst) { // Test Multi DataLayout graph cases, when the data layout changes after configure @@ -237,7 +239,7 @@ protected: ActivationLayerInfo _act_info{}; unsigned int _depth_multiplier{}; Size2D _dilation{}; - bool _mixed_layout{false}; + bool _mixed_layout{ false }; }; template @@ -309,6 +311,8 @@ public: void allocate_and_run_target() { + add_padding_x({ &_src, &_weights, &_biases, &_target }, _data_layout); + // Allocate tensors _src.allocator()->allocate(); _weights.allocator()->allocate(); @@ -442,6 +446,8 @@ public: void allocate_and_run_target() { + add_padding_x({ &_src, &_weights, &_biases, &_target }, _data_layout); + // Allocate tensors _src.allocator()->allocate(); _weights.allocator()->allocate(); -- cgit v1.2.1