Bug 1295811 - Remove check for __attribute__(aligned()); r=glandium
authorGregory Szorc <gps@mozilla.com>
Tue, 16 Aug 2016 21:14:42 -0700
changeset 309927 f904a6f393b6dfbd16dd4af439a2a0d5c11cd80c
parent 309926 6d227c4f173d3cf8365e1563a1ad7c3c64e93379
child 309928 e3a4670d7b6a86ba401122594aa39ffb5a2a9c9c
push id20342
push userkwierso@gmail.com
push dateThu, 18 Aug 2016 23:33:24 +0000
treeherderfx-team@abac53d4d247 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
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
@@ -3158,37 +3158,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"