From b0e687ef42e21b1eb7af18c4eaebcd41b0bd5632 Mon Sep 17 00:00:00 2001 From: Peter O'Gorman Date: Fri, 22 Aug 2008 18:18:25 +0200 Subject: [PATCH] Limit AC_C_BIGENDIAN univeral checks to Mac OS X. * lib/autoconf/c.m4 (AC_C_BIGENDIAN): Limit the check to __APPLE_CC__ with possible -arch flags. * NEWS: Document it. Signed-off-by: Ralf Wildenhues --- ChangeLog | 7 +++++++ NEWS | 4 ++++ lib/autoconf/c.m4 | 33 +++++++++++++++++++++++---------- 3 files changed, 34 insertions(+), 10 deletions(-) diff --git a/ChangeLog b/ChangeLog index be2e04c7d..5b4550404 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,10 @@ +2008-08-22 Peter O'Gorman + + Limit AC_C_BIGENDIAN univeral checks to Mac OS X. + * lib/autoconf/c.m4 (AC_C_BIGENDIAN): Limit the check to + __APPLE_CC__ with possible -arch flags. + * NEWS: Document it. + 2008-08-22 Ralf Wildenhues * NEWS: Fix typo. diff --git a/NEWS b/NEWS index 7fe8dac9c..ffeaf8bc9 100644 --- a/NEWS +++ b/NEWS @@ -2,6 +2,10 @@ GNU Autoconf NEWS - User visible changes. * Major changes in Autoconf 2.62a (2008-??-??) +** AC_C_BIGENDIAN does not mistakenly report "universal" for some + bigendian hosts, a regression introduced with universal binary + support in 2.62. + ** AC_PATH_X now includes /lib64 and /usr/lib64 in its list of default library directories. diff --git a/lib/autoconf/c.m4 b/lib/autoconf/c.m4 index ffaa06e39..51fdfeba6 100644 --- a/lib/autoconf/c.m4 +++ b/lib/autoconf/c.m4 @@ -1399,24 +1399,34 @@ fi AC_DEFUN([AC_C_BIGENDIAN], [AH_VERBATIM([WORDS_BIGENDIAN], [/* Define WORDS_BIGENDIAN to 1 if your processor stores words with the most - significant byte first (like Motorola and SPARC, unlike Intel and VAX). */ -#if defined __BIG_ENDIAN__ -# define WORDS_BIGENDIAN 1 -#elif ! defined __LITTLE_ENDIAN__ -# undef WORDS_BIGENDIAN + significant byte first (like Motorola and SPARC, unlike Intel). */ +#if defined AC_APPLE_UNIVERSAL_BUILD +# if defined __BIG_ENDIAN__ +# define WORDS_BIGENDIAN 1 +# endif +#else +# ifndef WORDS_BIGENDIAN +# undef WORDS_BIGENDIAN +# endif #endif])dnl AC_CACHE_CHECK([whether byte ordering is bigendian], [ac_cv_c_bigendian], [ac_cv_c_bigendian=unknown m4_ifval(m4_ifdef([AH_HEADER], 1)[$4], - [# See if __BIG_ENDIAN__ or __LITTLE_ENDIAN__ is defined. + [# See if we're dealing with a universal compiler. AC_COMPILE_IFELSE( [AC_LANG_SOURCE( - [[#if ! (defined __BIG_ENDIAN__ || defined __LITTLE_ENDIAN__) - neither is defined; + [[#ifndef __APPLE_CC__ + not a universal capable compiler #endif typedef int dummy; ]])], - [ac_cv_c_bigendian=universal])], + [ + # Check for potential -arch flags. It is not universal unless + # there are some -arch flags. Note that *ppc* also matches + # ppc64. This check is also rather less than ideal. + case "${CC} ${CFLAGS} ${CPPFLAGS} ${LDFLAGS}" in #( + *-arch*ppc*|*-arch*i386*|*-arch*x86_64*) ac_cv_c_bigendian=universal;; + esac])], [AC_DIAGNOSE([obsolete], [AC_C_BIGENDIAN suggests AC_CONFIG_HEADERS])]) if test $ac_cv_c_bigendian = unknown; then # See if sys/param.h defines the BYTE_ORDER macro. @@ -1521,7 +1531,10 @@ AC_DEFUN([AC_C_BIGENDIAN], no) $2 ;; #( universal) - $4 ;; #( + m4_default([$4], + [AC_DEFINE([AC_APPLE_UNIVERSAL_BUILD],1, + [Define if building universal (internal helper macro)])]) + ;; #( *) m4_default([$3], [AC_MSG_ERROR([unknown endianness -- 2.47.3