Fix redundancy in call to configure() in ACL DepthwiseConvolution workloads

Signed-off-by: Aron Virginas-Tar <Aron.Virginas-Tar@arm.com>
Change-Id: I8f698c6ec9826ce1188bc43bd59fbf7b83455c1a

diff --git a/src/backends/cl/workloads/ClDepthwiseConvolutionWorkload.cpp b/src/backends/cl/workloads/ClDepthwiseConvolutionWorkload.cpp
index eb837d7..858eab4 100644 (file)
--- a/src/backends/cl/workloads/ClDepthwiseConvolutionWorkload.cpp
+++ b/src/backends/cl/workloads/ClDepthwiseConvolutionWorkload.cpp
@@ -114,35 +114,16 @@ ClDepthwiseConvolutionWorkload::ClDepthwiseConvolutionWorkload(
 
     arm_compute::PadStrideInfo padStrideInfo = BuildArmComputePadStrideInfo(m_Data.m_Parameters);
 
-    // Check for optimisation opportunities.
-    bool use3x3Optimisation = (weightInfo.GetShape()[2] == 3) && (weightInfo.GetShape()[3] == 3);
-    if (use3x3Optimisation)
-    {
-        m_DepthwiseConvolutionLayer = std::make_unique<arm_compute::CLDepthwiseConvolutionLayer>();
-        static_cast<arm_compute::CLDepthwiseConvolutionLayer*>(m_DepthwiseConvolutionLayer.get())->configure(
-            &input,
-            m_KernelTensor.get(),
-            m_BiasTensor.get(),
-            &output,
-            padStrideInfo,
-            depthMultiplier,
-            arm_compute::ActivationLayerInfo(),
-            aclDilationInfo);
-    }
-    else
-    {
-        m_DepthwiseConvolutionLayer = std::make_unique<arm_compute::CLDepthwiseConvolutionLayer>();
-        static_cast<arm_compute::CLDepthwiseConvolutionLayer*>(m_DepthwiseConvolutionLayer.get())->configure(
-            &input,
-            m_KernelTensor.get(),
-            m_BiasTensor.get(),
-            &output,
-            padStrideInfo,
-            depthMultiplier,
-            arm_compute::ActivationLayerInfo(),
-            aclDilationInfo);
-
-    }
+    m_DepthwiseConvolutionLayer = std::make_unique<arm_compute::CLDepthwiseConvolutionLayer>();
+    static_cast<arm_compute::CLDepthwiseConvolutionLayer*>(m_DepthwiseConvolutionLayer.get())->configure(
+        &input,
+        m_KernelTensor.get(),
+        m_BiasTensor.get(),
+        &output,
+        padStrideInfo,
+        depthMultiplier,
+        arm_compute::ActivationLayerInfo(),
+        aclDilationInfo);
 
     BOOST_ASSERT(m_DepthwiseConvolutionLayer);
 

diff --git a/src/backends/neon/workloads/NeonDepthwiseConvolutionWorkload.cpp b/src/backends/neon/workloads/NeonDepthwiseConvolutionWorkload.cpp
index 2093613..f039a6b 100644 (file)
--- a/src/backends/neon/workloads/NeonDepthwiseConvolutionWorkload.cpp
+++ b/src/backends/neon/workloads/NeonDepthwiseConvolutionWorkload.cpp
@@ -113,48 +113,16 @@ NeonDepthwiseConvolutionWorkload::NeonDepthwiseConvolutionWorkload(
 
     arm_compute::PadStrideInfo padStrideInfo = BuildArmComputePadStrideInfo(m_Data.m_Parameters);
 
-    const arm_compute::ITensorInfo* inputInfo  = input.info();
-    const arm_compute::ITensorInfo* kernelInfo = m_KernelTensor->info();
-    const arm_compute::ITensorInfo* biasInfo   = m_BiasTensor ? m_BiasTensor->info() : nullptr;
-    const arm_compute::ITensorInfo* outputInfo = output.info();
-
-    // Check for optimisation opportunities
-    arm_compute::Status optimizationStatus =
-        arm_compute::NEDepthwiseConvolutionLayer::validate(inputInfo,
-                                                           kernelInfo,
-                                                           biasInfo,
-                                                           outputInfo,
-                                                           padStrideInfo,
-                                                           depthMultiplier,
-                                                           arm_compute::ActivationLayerInfo(),
-                                                           aclDilationInfo);
-
-    if (optimizationStatus.error_code() == arm_compute::ErrorCode::OK)
-    {
-        m_pDepthwiseConvolutionLayer = std::make_unique<arm_compute::NEDepthwiseConvolutionLayer>();
-        static_cast<arm_compute::NEDepthwiseConvolutionLayer*>(
-            m_pDepthwiseConvolutionLayer.get())->configure(&input,
-                                                           m_KernelTensor.get(),
-                                                           m_BiasTensor.get(),
-                                                           &output,
-                                                           padStrideInfo,
-                                                           depthMultiplier,
-                                                           arm_compute::ActivationLayerInfo(),
-                                                           aclDilationInfo);
-    }
-    else
-    {
-        m_pDepthwiseConvolutionLayer = std::make_unique<arm_compute::NEDepthwiseConvolutionLayer>();
-        static_cast<arm_compute::NEDepthwiseConvolutionLayer*>(
-            m_pDepthwiseConvolutionLayer.get())->configure(&input,
-                                                           m_KernelTensor.get(),
-                                                           m_BiasTensor.get(),
-                                                           &output,
-                                                           padStrideInfo,
-                                                           depthMultiplier,
-                                                           arm_compute::ActivationLayerInfo(),
-                                                           aclDilationInfo);
-    }
+    m_pDepthwiseConvolutionLayer = std::make_unique<arm_compute::NEDepthwiseConvolutionLayer>();
+    static_cast<arm_compute::NEDepthwiseConvolutionLayer*>(
+        m_pDepthwiseConvolutionLayer.get())->configure(&input,
+                                                        m_KernelTensor.get(),
+                                                        m_BiasTensor.get(),
+                                                        &output,
+                                                        padStrideInfo,
+                                                        depthMultiplier,
+                                                        arm_compute::ActivationLayerInfo(),
+                                                        aclDilationInfo);
 
     BOOST_ASSERT(m_pDepthwiseConvolutionLayer);