On 9/26/25 14:40, David Marchand wrote:
This was caught while checking ARM arch headers.
In file included from buildtools/chkincs/chkincs-cpp.p/rte_memcpy_32.cpp:1:
/home/runner/work/dpdk/dpdk/lib/eal/arm/include/rte_memcpy_32.h:302:1:
error: expected declaration before ‘}’ token
302 | }
| ^
Acked-by: Mattias Rönnblom <[email protected]>
Fixes: 719834a6849e ("use C linkage where appropriate in headers")
Cc: [email protected]
Signed-off-by: David Marchand <[email protected]>
---
lib/eal/arm/include/rte_memcpy_32.h | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/lib/eal/arm/include/rte_memcpy_32.h
b/lib/eal/arm/include/rte_memcpy_32.h
index 99fd5757ca..861be06f5e 100644
--- a/lib/eal/arm/include/rte_memcpy_32.h
+++ b/lib/eal/arm/include/rte_memcpy_32.h
@@ -19,10 +19,14 @@
/* ARM NEON Intrinsics are used to copy data */
#include <arm_neon.h>
+#endif /* RTE_ARCH_ARM_NEON_MEMCPY */
+
#ifdef __cplusplus
extern "C" {
#endif
+#ifdef RTE_ARCH_ARM_NEON_MEMCPY
+
static inline void
rte_mov16(uint8_t *dst, const uint8_t *src)
{
@@ -252,7 +256,7 @@ rte_memcpy_func(void *dst, const void *src, size_t n)
return ret;
}
-#else
+#else /* ! RTE_ARCH_ARM_NEON_MEMCPY */
static inline void
rte_mov16(uint8_t *dst, const uint8_t *src)
