Re: [dpdk-stable] [PATCH] eal: fix build of external apps with clang on armv8

[Thomas Monjalon <thomas@monjalon.net>]

14/01/2019 17:27, Thomas Monjalon:
> 14/01/2019 17:14, Ilya Maximets:
> > In case DPDK built using GCC, RTE_TOOLCHAIN_CLANG is not defined.
> > But 'rte_atomic.h' is a generic header that included to the
> > external apps like OVS while building with DPDK. As a result,
> > clang build of OVS fails on ARMv8 if DPDK built using gcc:
> > 
> >     include/generic/rte_atomic.h:215:9: error:
> >             implicit declaration of function '__atomic_exchange_2'
> >             is invalid in C99
> >     include/generic/rte_atomic.h:494:9: error:
> >             implicit declaration of function '__atomic_exchange_4'
> >             is invalid in C99
> >     include/generic/rte_atomic.h:772:9: error:
> >             implicit declaration of function '__atomic_exchange_8'
> >             is invalid in C99
> > 
> > We need to check for current compiler, not the compiler used for
> > DPDK build.
> 
> Right, API cannot rely on internal build system configuration.
> 
> > Fixes: 7bdccb93078e ("eal: fix ARM build with clang")
> > Cc: stable@dpdk.org
> > 
> > Signed-off-by: Ilya Maximets <i.maximets@samsung.com>
> 
> Acked-by: Thomas Monjalon <thomas@monjalon.net>

Applied (if armv8 typo), thanks