From 8df95843ed1090b7be41b541bbcf111b86cb4c4d Mon Sep 17 00:00:00 2001 From: Thomas Petazzoni Date: Tue, 2 Feb 2016 16:31:33 +0100 Subject: [PATCH] glog: fix atomic built-in problem glog can optionally use atomic __sync built-ins. However, its configure script only checks for the availability of the 4-byte variant, but the code also uses the 1-byte variant. While this works on most architectures, it does not on architectures that implement only the 4-byte variant, such as Microblaze. So if the architecture does not implement the 1-byte variant, we hint the configure script that atomic built-ins should not be used. Fixes: http://autobuild.buildroot.org/results/bedebd594c283adf39374c9a9f44432e5d6cbff7/ Signed-off-by: Thomas Petazzoni Reviewed-by: "Yann E. MORIN" --- package/glog/glog.mk | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/package/glog/glog.mk b/package/glog/glog.mk index 8a8409487a..aa7cacbdd3 100644 --- a/package/glog/glog.mk +++ b/package/glog/glog.mk @@ -14,4 +14,15 @@ ifeq ($(BR2_PACKAGE_GFLAGS),y) GLOG_DEPENDENCIES = gflags endif +# glog can optionally use atomic __sync built-ins. However, its +# configure script only checks for the availability of the 4 bytes +# version, but the code also uses the 1 byte version. While this works +# on most architectures, it does not on architectures that implement +# only the 4 bytes version, such as Microblaze. So if the architecture +# does not implement the 1 byte version, we hint the configure script +# that atomic built-ins should not be used. +ifeq ($(BR2_TOOLCHAIN_HAS_SYNC_1),) +GLOG_CONF_ENV += ac_cv___sync_val_compare_and_swap=no +endif + $(eval $(autotools-package))