diff options
| author | Anthony Barbier <anthony.barbier@arm.com> | 2018-01-11 10:45:24 +0000 |
|---|---|---|
| committer | Anthony Barbier <anthony.barbier@arm.com> | 2018-11-02 16:43:10 +0000 |
| commit | 72856abfd5c0cd24542bd2ec4f26785d99683c7b (patch) | |
| tree | 93ff9d30d868013419e83ee2592149716e6aca7c /tests/benchmark/fixtures/ConvolutionLayerFixture.h | |
| parent | 2b5f0f2574551f59970bb9d710bafad2bc4bbd4a (diff) | |
| download | ComputeLibrary-72856abfd5c0cd24542bd2ec4f26785d99683c7b.tar.gz | |
COMPMID-786: Remove all the fill() calls in benchmarks
Filling buffers with random data takes a significant amount of time and in most cases doesn't affect the performance
We will therefore only keep fill() in the functions for which it matters
Change-Id: Ica34fe09941f27d6f0417f33176847febf722bc3
Reviewed-on: https://eu-gerrit-1.euhpc.arm.com/115892
Tested-by: Jenkins <bsgcomp@arm.com>
Reviewed-by: Gian Marco Iodice <gianmarco.iodice@arm.com>
Diffstat (limited to 'tests/benchmark/fixtures/ConvolutionLayerFixture.h')
| -rw-r--r-- | tests/benchmark/fixtures/ConvolutionLayerFixture.h | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/tests/benchmark/fixtures/ConvolutionLayerFixture.h b/tests/benchmark/fixtures/ConvolutionLayerFixture.h index 403160782d..0d2c3fd955 100644 --- a/tests/benchmark/fixtures/ConvolutionLayerFixture.h +++ b/tests/benchmark/fixtures/ConvolutionLayerFixture.h @@ -1,5 +1,5 @@ /* - * Copyright (c) 2017 ARM Limited. + * Copyright (c) 2017-2018 ARM Limited. * * SPDX-License-Identifier: MIT * @@ -61,11 +61,6 @@ public: weights.allocator()->allocate(); biases.allocator()->allocate(); dst.allocator()->allocate(); - - // Fill tensors - library->fill_tensor_uniform(Accessor(src), 0); - library->fill_tensor_uniform(Accessor(weights), 1); - library->fill_tensor_uniform(Accessor(biases), 2); } void run() |