Bug 1191900 - remove superfluous check for GCC force_align_arg_pointer attribute; r=glandium

qcms and libav use __attribute__((force_align_arg_pointer))
unconditionally; the libav use case suggests that the attribute has been
around since GCC 4.2.  We're well past that point with GCC, and clang
supports it also.  So we can simply assume the compiler has it in the
appropriate places.

It is, however, x86 only (x86-64 appropriately aligns the stack at all
times), so we need to adjust the libpixman build code appropriately.
This commit is contained in:
Nathan Froyd 2015-08-06 21:07:57 -04:00
parent 0a5095ccbf
commit 7894c2fd13
2 changed files with 2 additions and 28 deletions

View File

@ -2645,26 +2645,6 @@ AC_SUBST(VISIBILITY_FLAGS)
MOZ_LLVM_PR8927
dnl Check for __force_align_arg_pointer__ for SSE2 on gcc
dnl ========================================================
if test "$GNU_CC"; then
CFLAGS_save="${CFLAGS}"
CFLAGS="${CFLAGS} -Werror"
AC_CACHE_CHECK(for __force_align_arg_pointer__ attribute,
ac_cv_force_align_arg_pointer,
[AC_TRY_COMPILE([__attribute__ ((__force_align_arg_pointer__)) void test() {}],
[],
ac_cv_force_align_arg_pointer="yes",
ac_cv_force_align_arg_pointer="no")])
CFLAGS="${CFLAGS_save}"
if test "$ac_cv_force_align_arg_pointer" = "yes"; then
HAVE_GCC_ALIGN_ARG_POINTER=1
else
HAVE_GCC_ALIGN_ARG_POINTER=
fi
fi
AC_SUBST(HAVE_GCC_ALIGN_ARG_POINTER)
dnl Checks for header files.
dnl ========================================================
AC_HEADER_DIRENT

View File

@ -79,18 +79,12 @@ use_vmx = False
use_arm_simd_gcc = False
use_arm_neon_gcc = False
if '86' in CONFIG['OS_TEST']:
if '64' in CONFIG['OS_TEST']:
if CONFIG['GNU_CC']:
use_sse2 = True
else:
use_sse2 = True
if '64' not in CONFIG['OS_TEST']:
if CONFIG['_MSC_VER']:
use_mmx = True
if CONFIG['HAVE_GCC_ALIGN_ARG_POINTER']:
use_sse2 = True
if CONFIG['GNU_CC']:
use_mmx = True
if CONFIG['_MSC_VER']:
use_sse2 = True
elif 'ppc' in CONFIG['OS_TEST']:
if CONFIG['GNU_CC']:
use_vmx = True