[lng-odp] [PATCH] build: fix 64-bit atomics detection
Use uint64_t instead of int type. This resolves ipfragreass build breakage with clang on 32-bit systems. Signed-off-by: Brian Brooks --- platform/linux-generic/m4/configure.m4 | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/platform/linux-generic/m4/configure.m4 b/platform/linux-generic/m4/configure.m4 index e1197f60..c8164b8c 100644 --- a/platform/linux-generic/m4/configure.m4 +++ b/platform/linux-generic/m4/configure.m4 @@ -34,10 +34,11 @@ use_libatomic=no AC_MSG_CHECKING(whether -latomic is needed for 64-bit atomic built-ins) AC_LINK_IFELSE( [AC_LANG_SOURCE([[ -static int loc; +#include +static uint64_t loc; int main(void) { -int prev = __atomic_exchange_n(&loc, 7, __ATOMIC_RELAXED); +uint64_t prev = __atomic_exchange_n(&loc, 7, __ATOMIC_RELAXED); return 0; } ]])], -- 2.13.1
Re: [lng-odp] [PATCH] build: fix 64-bit atomics detection
On 27 June 2017 at 22:58, Brian Brooks wrote: > Use uint64_t instead of int type. > > This resolves ipfragreass build breakage with clang on 32-bit systems. > > Change-Id: I23b9b6a46cbfaeff99116dac9c22e86629cbdf24 > looks ok for me, but please remove change-id and add sign-off. Maxim. > --- > platform/linux-generic/m4/configure.m4 | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/platform/linux-generic/m4/configure.m4 > b/platform/linux-generic/m4/configure.m4 > index e1197f60..c8164b8c 100644 > --- a/platform/linux-generic/m4/configure.m4 > +++ b/platform/linux-generic/m4/configure.m4 > @@ -34,10 +34,11 @@ use_libatomic=no > AC_MSG_CHECKING(whether -latomic is needed for 64-bit atomic built-ins) > AC_LINK_IFELSE( >[AC_LANG_SOURCE([[ > -static int loc; > +#include > +static uint64_t loc; > int main(void) > { > -int prev = __atomic_exchange_n(&loc, 7, __ATOMIC_RELAXED); > +uint64_t prev = __atomic_exchange_n(&loc, 7, __ATOMIC_RELAXED); > return 0; > } > ]])], > -- > 2.13.1 > >
[lng-odp] [PATCH] build: fix 64-bit atomics detection
Use uint64_t instead of int type. This resolves ipfragreass build breakage with clang on 32-bit systems. Change-Id: I23b9b6a46cbfaeff99116dac9c22e86629cbdf24 --- platform/linux-generic/m4/configure.m4 | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/platform/linux-generic/m4/configure.m4 b/platform/linux-generic/m4/configure.m4 index e1197f60..c8164b8c 100644 --- a/platform/linux-generic/m4/configure.m4 +++ b/platform/linux-generic/m4/configure.m4 @@ -34,10 +34,11 @@ use_libatomic=no AC_MSG_CHECKING(whether -latomic is needed for 64-bit atomic built-ins) AC_LINK_IFELSE( [AC_LANG_SOURCE([[ -static int loc; +#include +static uint64_t loc; int main(void) { -int prev = __atomic_exchange_n(&loc, 7, __ATOMIC_RELAXED); +uint64_t prev = __atomic_exchange_n(&loc, 7, __ATOMIC_RELAXED); return 0; } ]])], -- 2.13.1