Bug 1295811 - Remove check for __attribute__(aligned()); r?glandium draft
authorGregory Szorc <gps@mozilla.com>
Tue, 16 Aug 2016 21:14:42 -0700
changeset 401561 12c053b18d8f6383f9fe3ea576ca0817206eaa96
parent 401560 8aa861efa70b64ba52acabb693da00f81927ed69
child 528513 44b09f72266ea9fdfc613c75b3266a2531d439cd
push id26486
push usergszorc@mozilla.com
push dateWed, 17 Aug 2016 04:16:40 +0000
reviewersglandium
bugs1295811, 556455
milestone51.0a1
Bug 1295811 - Remove check for __attribute__(aligned()); r?glandium It was setting ATTRIBUTE_ALIGNED_MAX which hasn't been used for 6 years since 590a7be0e60f (bug 556455) removed its only consumer. MozReview-Commit-ID: HSAql5cUrQH
old-configure.in
--- a/old-configure.in
+++ b/old-configure.in
@@ -3165,37 +3165,16 @@ MOZ_ARG_ENABLE_BOOL(raw,
     MOZ_RAW=)
 
 if test -n "$MOZ_RAW"; then
     AC_DEFINE(MOZ_RAW)
 fi
 
 AC_SUBST(MOZ_RAW)
 
-dnl Checks for __attribute__(aligned()) directive need by libogg
-AC_CACHE_CHECK([__attribute__ ((aligned ())) support],
-    [ac_cv_c_attribute_aligned],
-    [ac_cv_c_attribute_aligned=0
-     CFLAGS_save="${CFLAGS}"
-     CFLAGS="${CFLAGS} -Werror"
-     for ac_cv_c_attr_align_try in 64 32 16 8; do
-       echo "trying $ac_cv_c_attr_align_try"
-       AC_TRY_COMPILE([],
-                      [static char c __attribute__ ((aligned(${ac_cv_c_attr_align_try}))) = 0; return c;],
-                      [ac_cv_c_attribute_aligned="${ac_cv_c_attr_align_try}"])
-       if test "$ac_cv_c_attribute_aligned" != 0; then
-         break;
-       fi
-     done
-       CFLAGS="${CFLAGS_save}"])
-if test "${ac_cv_c_attribute_aligned}" != "0"; then
-  AC_DEFINE_UNQUOTED([ATTRIBUTE_ALIGNED_MAX],
-                     [${ac_cv_c_attribute_aligned}],[Maximum supported data alignment])
-fi
-
 dnl ========================================================
 dnl = Apple platform decoder support
 dnl ========================================================
 if test "$COMPILE_ENVIRONMENT"; then
 if test -n "$MOZ_APPLEMEDIA"; then
   # hack in frameworks for fmp4 - see bug 1029974
   # We load VideoToolbox and CoreMedia dynamically, so they don't appear here.
   LDFLAGS="$LDFLAGS -framework AudioToolbox"