[lng-odp] [PATCH] build: fix 64-bit atomics detection

2017-06-27 Thread Brian Brooks
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(, 7, __ATOMIC_RELAXED);
+uint64_t prev = __atomic_exchange_n(, 7, __ATOMIC_RELAXED);
 return 0;
 }
 ]])],
-- 
2.13.1



Re: [lng-odp] [PATCH] build: fix 64-bit atomics detection

2017-06-27 Thread Maxim Uvarov
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(, 7, __ATOMIC_RELAXED);
> +uint64_t prev = __atomic_exchange_n(, 7, __ATOMIC_RELAXED);
>  return 0;
>  }
>  ]])],
> --
> 2.13.1
>
>