commit 4646d2e026a0fa92085fcba2f4aec5ec148956aa
author Michele Di Giorgio <michele.digiorgio@arm.com> Wed Jun 19 12:28:47 2019 +0100
committer Michele Di Giorgio <michele.digiorgio@arm.com> Wed Jul 03 09:36:46 2019 +0000
tree 0847684b138f9b4631cf56e20d0e75ef313607d0
parent cbe39055f61bc7a3f6aef4588de47d48b77cf354

COMPMID-2072: Use a constexpr for num_elems_processed_per_iteration where possible

Change-Id: I26cb699ae3a77003ef7d05ac30d3ed518214e25f
Signed-off-by: Michele Di Giorgio <michele.digiorgio@arm.com>
Reviewed-on: https://review.mlplatform.org/c/1375
Comments-Addressed: Arm Jenkins <bsgcomp@arm.com>
Reviewed-by: Manuel Bottini <manuel.bottini@arm.com>
Tested-by: Arm Jenkins <bsgcomp@arm.com>
Reviewed-by: Pablo Marquez <pablo.tello@arm.com>

src/core/CL/kernels/CLAccumulateKernel.cpp

diff --git a/src/core/CL/kernels/CLAccumulateKernel.cpp b/src/core/CL/kernels/CLAccumulateKernel.cpp
index 6333f04..12ee210 100644
--- a/src/core/CL/kernels/CLAccumulateKernel.cpp
+++ b/src/core/CL/kernels/CLAccumulateKernel.cpp
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2016, 2017 ARM Limited.
+ * Copyright (c) 2016-2019 ARM Limited.
  *
  * SPDX-License-Identifier: MIT
  *
@@ -31,7 +31,12 @@
 #include "arm_compute/core/Types.h"
 #include "arm_compute/core/Validate.h"
 
-using namespace arm_compute;
+namespace arm_compute
+{
+namespace
+{
+constexpr unsigned int num_elems_processed_per_iteration = 16;
+} // namespace
 
 void CLAccumulateKernel::configure(const ICLTensor *input, ICLTensor *accum)
 {
@@ -42,7 +47,6 @@
     _kernel = static_cast<cl::Kernel>(CLKernelLibrary::get().create_kernel("accumulate"));
 
     // Make sure _kernel is initialized before calling the parent's configure
-    constexpr unsigned int num_elems_processed_per_iteration = 16;
     ICLSimple2DKernel::configure(input, accum, num_elems_processed_per_iteration);
 }
 
@@ -60,7 +64,6 @@
     _kernel.setArg(idx++, alpha);
 
     // Configure kernel window
-    constexpr unsigned int num_elems_processed_per_iteration = 16;
     ICLSimple2DKernel::configure(input, accum, num_elems_processed_per_iteration);
 }
 
@@ -78,6 +81,6 @@
     _kernel.setArg(idx++, shift);
 
     // Configure kernel window
-    constexpr unsigned int num_elems_processed_per_iteration = 16;
     ICLSimple2DKernel::configure(input, accum, num_elems_processed_per_iteration);
 }
+} // namespace arm_compute