Revise acl neon backend cmake (#6393)

- Set strict build option: nnfw_common
- Remove needless link
- Update header include for acl common

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>

diff --git a/runtimes/neurun/backend/acl_neon/CMakeLists.txt b/runtimes/neurun/backend/acl_neon/CMakeLists.txt
index 0ffcc6b..3fb66a0 100644 (file)
--- a/runtimes/neurun/backend/acl_neon/CMakeLists.txt
+++ b/runtimes/neurun/backend/acl_neon/CMakeLists.txt
@@ -10,18 +10,11 @@ file(GLOB_RECURSE SOURCES "*.cc")
 
 add_library(${LIB_NEURUN_BACKEND_ACL_NEON} SHARED ${SOURCES})
 
-target_include_directories(${LIB_NEURUN_BACKEND_ACL_NEON} PUBLIC ${NEURUN_INCLUDE_DIR})
 target_include_directories(${LIB_NEURUN_BACKEND_ACL_NEON} PRIVATE ${CMAKE_CURRENT_SOURCE_DIR})
-#following line it to reuse codes of other backends
-target_include_directories(${LIB_NEURUN_BACKEND_ACL_NEON} PUBLIC ${CMAKE_CURRENT_SOURCE_DIR}/..)
-
-target_link_libraries(${LIB_NEURUN_BACKEND_ACL_NEON} arm_compute)
-target_link_libraries(${LIB_NEURUN_BACKEND_ACL_NEON} arm_compute_ex)
-target_link_libraries(${LIB_NEURUN_BACKEND_ACL_NEON} neurun-core)
-
-target_link_libraries(${LIB_NEURUN_BACKEND_ACL_NEON} ${LIB_NEURUN_BACKEND_ACL_COMMON})
-
-target_compile_options(${LIB_NEURUN_BACKEND_ACL_NEON} PRIVATE -Wall -Wextra -Werror)
+target_link_libraries(${LIB_NEURUN_BACKEND_ACL_NEON} PRIVATE neurun-core)
+target_link_libraries(${LIB_NEURUN_BACKEND_ACL_NEON} PRIVATE ${LIB_NEURUN_BACKEND_ACL_COMMON})
+target_link_libraries(${LIB_NEURUN_BACKEND_ACL_NEON} PRIVATE nnfw_common)
 
 set_target_properties(${LIB_NEURUN_BACKEND_ACL_NEON} PROPERTIES OUTPUT_NAME backend_acl_neon)
+
 install(TARGETS ${LIB_NEURUN_BACKEND_ACL_NEON} DESTINATION lib)

diff --git a/runtimes/neurun/backend/acl_neon/KernelGenerator.cc b/runtimes/neurun/backend/acl_neon/KernelGenerator.cc
index 72be6bd..0b14e41 100644 (file)
--- a/runtimes/neurun/backend/acl_neon/KernelGenerator.cc
+++ b/runtimes/neurun/backend/acl_neon/KernelGenerator.cc
@@ -33,6 +33,9 @@
 #include <arm_compute/runtime/NEON/functions/NEFullyConnectedReshapingLayer.h>
 #include <arm_compute/runtime/NEON/functions/NETransposeConvLayer.h>
 
+#include <Convert.h>
+#include <Swizzle.h>
+
 #include "kernel/ConcatLayer.h"
 #include "util/Padding.h"
 #include "model/Index.h"
@@ -42,8 +45,6 @@
 #include "exec/NopFunction.h"
 #include "util/logging.h"
 #include "util/Utils.h"
-#include "acl_common/Convert.h"
-#include "acl_common/Swizzle.h"
 
 using ::neurun::compiler::IExecutionBuilder;
 

diff --git a/runtimes/neurun/backend/acl_neon/MemoryManager.h b/runtimes/neurun/backend/acl_neon/MemoryManager.h
index f6d2689..b47cd66 100644 (file)
--- a/runtimes/neurun/backend/acl_neon/MemoryManager.h
+++ b/runtimes/neurun/backend/acl_neon/MemoryManager.h
@@ -23,8 +23,8 @@
 #include <arm_compute/runtime/MemoryManagerOnDemand.h>
 #include <arm_compute/runtime/MemoryGroup.h>
 
-#include "acl_common/AclMemoryManager.h"
-#include "acl_common/AclLinearMemoryManager.h"
+#include <AclMemoryManager.h>
+#include <AclLinearMemoryManager.h>
 
 #include "operand/NETensor.h"
 #include "operand/NESubTensor.h"

diff --git a/runtimes/neurun/backend/acl_neon/ShapeFixer.cc b/runtimes/neurun/backend/acl_neon/ShapeFixer.cc
index 5eda116..e6fdca7 100644 (file)
--- a/runtimes/neurun/backend/acl_neon/ShapeFixer.cc
+++ b/runtimes/neurun/backend/acl_neon/ShapeFixer.cc
@@ -28,6 +28,9 @@
 #include <arm_compute/runtime/NEON/functions/NEFullyConnectedLayer.h>
 #include <arm_compute/runtime/NEON/functions/NEFullyConnectedReshapingLayer.h>
 
+#include <Convert.h>
+#include <Swizzle.h>
+
 #include "kernel/ConcatLayer.h"
 #include "util/Padding.h"
 #include "model/Index.h"
@@ -37,8 +40,6 @@
 #include "exec/NopFunction.h"
 #include "util/logging.h"
 #include "util/Utils.h"
-#include "acl_common/Convert.h"
-#include "acl_common/Swizzle.h"
 
 using ::neurun::compiler::IExecutionBuilder;
 

diff --git a/runtimes/neurun/backend/acl_neon/TensorBuilder.h b/runtimes/neurun/backend/acl_neon/TensorBuilder.h
index b01971e..3da30df 100644 (file)
--- a/runtimes/neurun/backend/acl_neon/TensorBuilder.h
+++ b/runtimes/neurun/backend/acl_neon/TensorBuilder.h
@@ -17,9 +17,10 @@
 #ifndef __NEURUN_BACKEND_ACL_NEON_TENSOR_BUILDER_H__
 #define __NEURUN_BACKEND_ACL_NEON_TENSOR_BUILDER_H__
 
+#include <TemplTensorBuilder.h>
+
 #include <backend/operand/Object.h>
 
-#include "acl_common/TemplTensorBuilder.h"
 #include "operand/NETensor.h"
 #include "operand/NESubTensor.h"
 

diff --git a/runtimes/neurun/backend/acl_neon/kernel/ConcatLayer.cc b/runtimes/neurun/backend/acl_neon/kernel/ConcatLayer.cc
index 49ffa7b..d321fec 100644 (file)
--- a/runtimes/neurun/backend/acl_neon/kernel/ConcatLayer.cc
+++ b/runtimes/neurun/backend/acl_neon/kernel/ConcatLayer.cc
@@ -16,8 +16,9 @@
 
 #include "ConcatLayer.h"
 
+#include <Swizzle.h>
+
 #include "util/feature/nchw/View.h"
-#include "acl_common/Swizzle.h"
 #include "util/logging.h"
 
 namespace

diff --git a/runtimes/neurun/backend/acl_neon/operand/INETensor.h b/runtimes/neurun/backend/acl_neon/operand/INETensor.h
index 23e802d..256806a 100644 (file)
--- a/runtimes/neurun/backend/acl_neon/operand/INETensor.h
+++ b/runtimes/neurun/backend/acl_neon/operand/INETensor.h
@@ -18,7 +18,8 @@
 #define __NEURUN_BACKEND_ACL_NEON_OPERAND_I_NE_TENSOR_H__
 
 #include <arm_compute/core/ITensor.h>
-#include <acl_common/IACLTensor.h>
+
+#include <IACLTensor.h>
 
 namespace neurun
 {