[PATCH v5 6/9] eal/arm: fix C++ build for 32-bit memcpy
Mattias Rönnblom
hofors at lysator.liu.se
Sat Sep 27 08:33:55 CEST 2025
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 <mattias.ronnblom at ericsson.com>
> Fixes: 719834a6849e ("use C linkage where appropriate in headers")
> Cc: stable at dpdk.org
>
> Signed-off-by: David Marchand <david.marchand at redhat.com>
> ---
> 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)
More information about the dev
mailing list