commit a2b131bf4680f83b10a7e3544b6183279d8c2691
author SiCong Li <sicong.li@arm.com> Fri Nov 04 10:11:32 2022 +0000
committer SiCong Li <sicong.li@arm.com> Fri Nov 04 13:47:35 2022 +0000
tree a39f36d4e330ac5c0ff36feffc809224d159854f
parent d7c5d7d3635e43658123a0a5b2c1b77c81b9c26d

Fix compiler warnings in dynamic fusion

Resolves: COMPMID-5686

Change-Id: I608c359583c44f2f04f29faddd1c6b38a381de60
Signed-off-by: SiCong Li <sicong.li@arm.com>
Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/8562
Benchmark: Arm Jenkins <bsgcomp@arm.com>
Reviewed-by: Jakub Sujak <jakub.sujak@arm.com>
Reviewed-by: Gunes Bayir <gunes.bayir@arm.com>
Comments-Addressed: Arm Jenkins <bsgcomp@arm.com>
Tested-by: Arm Jenkins <bsgcomp@arm.com>

src/dynamic_fusion/runtime/gpu/cl/ClKernelRuntime.h

diff --git a/src/dynamic_fusion/runtime/gpu/cl/ClKernelRuntime.h b/src/dynamic_fusion/runtime/gpu/cl/ClKernelRuntime.h
index acc2380..4787aca 100644
--- a/src/dynamic_fusion/runtime/gpu/cl/ClKernelRuntime.h
+++ b/src/dynamic_fusion/runtime/gpu/cl/ClKernelRuntime.h
@@ -35,7 +35,8 @@
 {
 namespace dynamic_fusion
 {
-struct GpuKernelSourceCode;
+/** Forward declaration */
+class GpuKernelSourceCode;
 
 /** OpenCL runtime to run a single kernel */
 class ClKernelRuntime final : public opencl::IClKernel

src/dynamic_fusion/runtime/gpu/cl/ClWorkloadRuntime.cpp

diff --git a/src/dynamic_fusion/runtime/gpu/cl/ClWorkloadRuntime.cpp b/src/dynamic_fusion/runtime/gpu/cl/ClWorkloadRuntime.cpp
index 549c6d4..f14f66d 100644
--- a/src/dynamic_fusion/runtime/gpu/cl/ClWorkloadRuntime.cpp
+++ b/src/dynamic_fusion/runtime/gpu/cl/ClWorkloadRuntime.cpp
@@ -287,15 +287,20 @@
         switch(stage)
         {
             case UnitWorkloadStage::Stage::Run:
+            {
                 _impl->_kernels.emplace(work.id(), std::move(k));
                 break;
+            }
             case UnitWorkloadStage::Stage::Prepare:
+            {
                 _impl->_kernels_prep.emplace(work.id(), std::move(k));
                 break;
+            }
             default:
+            {
                 ARM_COMPUTE_ERROR("Invalid unit workload stage");
+            }
         }
-        break;
     }
     // Create auxiliary tensor objects
     create_aux_tensors(&_impl->_aux_tensors, _impl->_source_code);

src/dynamic_fusion/sketch/gpu/operators/GpuConv2d.cpp

diff --git a/src/dynamic_fusion/sketch/gpu/operators/GpuConv2d.cpp b/src/dynamic_fusion/sketch/gpu/operators/GpuConv2d.cpp
index 98c1cc3..12aa4d1 100644
--- a/src/dynamic_fusion/sketch/gpu/operators/GpuConv2d.cpp
+++ b/src/dynamic_fusion/sketch/gpu/operators/GpuConv2d.cpp
@@ -25,7 +25,6 @@
 
 #include "arm_compute/core/CL/CLCompileContext.h"
 #include "arm_compute/core/Validate.h"
-#include "arm_compute/core/Validate.h"
 #include "arm_compute/core/experimental/Types.h"
 #include "arm_compute/core/utils/misc/ShapeCalculator.h"
 
@@ -87,7 +86,7 @@
     return true;
 }
 
-GpuOperatorType operator_type = GpuOperatorType::Complex;
+constexpr GpuOperatorType operator_type = GpuOperatorType::Complex;
 } // namespace
 
 Status GpuConv2d::validate_op(const GpuWorkloadSketch &sketch,