commit 9a9440ebc0d6710d832abd1ba5daea5af08a5e5c
author Manuel Bottini <manuel.bottini@arm.com> Mon Jan 13 11:40:48 2020 +0000
committer Michele Di Giorgio <michele.digiorgio@arm.com> Mon Jan 13 18:12:05 2020 +0000
tree 5455e969789365cf3daeccaeb7b0c6f3ef7bac07
parent ec00da149b7884114cd6c43cc9c1cff62ddaa710

COMPMID-3005: Test error: CL/DeconvolutionLayer/Quantized/QASYMM8/W3x3

When assert is set to 0 the output was not initialized

Change-Id: Icf894d021f2b8932dabc55ae31c1195623be9e33
Signed-off-by: Manuel Bottini <manuel.bottini@arm.com>
Reviewed-on: https://review.mlplatform.org/c/2574
Tested-by: Arm Jenkins <bsgcomp@arm.com>
Reviewed-by: Michele Di Giorgio <michele.digiorgio@arm.com>

src/core/CL/kernels/CLGEMMLowpQuantizeDownInt32ToInt8ScaleByFixedPointKernel.cpp

diff --git a/src/core/CL/kernels/CLGEMMLowpQuantizeDownInt32ToInt8ScaleByFixedPointKernel.cpp b/src/core/CL/kernels/CLGEMMLowpQuantizeDownInt32ToInt8ScaleByFixedPointKernel.cpp
index 3de3182..8a5ce9f 100644
--- a/src/core/CL/kernels/CLGEMMLowpQuantizeDownInt32ToInt8ScaleByFixedPointKernel.cpp
+++ b/src/core/CL/kernels/CLGEMMLowpQuantizeDownInt32ToInt8ScaleByFixedPointKernel.cpp
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2019 ARM Limited.
+ * Copyright (c) 2019-2020 ARM Limited.
  *
  * SPDX-License-Identifier: MIT
  *
@@ -122,10 +122,9 @@
     // Perform validate step
     ARM_COMPUTE_ERROR_ON_NULLPTR(input, output);
     ARM_COMPUTE_ERROR_THROW_ON(validate_arguments(input->info(), (bias != nullptr) ? bias->info() : nullptr, output->info(), min, max));
-    ARM_COMPUTE_ERROR_THROW_ON(validate_and_configure_window(input->info(),
-                                                             (bias != nullptr) ? bias->info() : nullptr,
-                                                             output->info())
-                               .first);
+    // Configure kernel window
+    auto win_config = validate_and_configure_window(input->info(), (bias != nullptr) ? bias->info() : nullptr, output->info());
+    ARM_COMPUTE_ERROR_THROW_ON(win_config.first);
 
     _input  = input;
     _bias   = bias;
@@ -144,9 +143,6 @@
     // Create kernel
     _kernel = static_cast<cl::Kernel>(CLKernelLibrary::get().create_kernel("gemmlowp_output_stage_quantize_down_fixedpoint", build_opts.options()));
 
-    // Configure kernel window
-    auto win_config = validate_and_configure_window(input->info(), (bias != nullptr) ? bias->info() : nullptr, output->info());
-    ARM_COMPUTE_ERROR_THROW_ON(win_config.first);
     ICLKernel::configure_internal(win_config.second);
 }