Remove --with-tuning

With the wide range of deployed 686 processors, -mtune=generic is saner
and it is the default anyway. The VLC defaults did not even make sense
anymore (Pentium 2, seriously?).

You might need to set -march explicitly as with other build systems,
e.g.: -march=i686

diff --git a/NEWS b/NEWS
index 677e1ccadfbb1734dccdfe2ec236b51973ae7b9e..27eb933fbc9f9e7eff55a3d9ddb502b85cc15bdb 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -1,6 +1,10 @@
 Changes between 2.0.x and 2.1.0-git:
 --------------------------------
 
+Important changes for packagers:
+ * The --with-tuning and --without-tuning options are removed. To tune
+   compilation for a specific processor type, set CFLAGS manually.
+
 3rd party libraries (contrib):
  * /extras/contrib has been replaced by a better system in /contrib
 

diff --git a/configure.ac b/configure.ac
index fb83def61db82f57bccd9a0fcd11a56ba97e0e5e..8765b147a1be2f2bf680fb8188c972a45f7ef846 100644 (file)
--- a/configure.ac
+++ b/configure.ac
@@ -1481,44 +1481,6 @@ dnl - Others: test should fail
 ])
 AM_CONDITIONAL([HAVE_ALTIVEC], [test "$have_altivec" = "yes"])
 
-dnl
-dnl  Special arch tuning
-dnl
-AC_ARG_WITH(tuning,
-  [AS_HELP_STRING([--with-tuning=ARCH],
-    [tune compilation for an architecture (default varies)])])
-if test -n "${with_tuning}"; then
-    if test "${with_tuning}" != "no"; then
-        CFLAGS_TUNING="-mtune=${with_tuning}"
-    fi
-else
-    if test "${SYS}" = "darwin" -a "${host_cpu}" = "i686"; then
-        CFLAGS_TUNING="-march=prescott -mtune=generic"
-    elif test "${SYS}" = "darwin" -a "${host_cpu}" = "x86_64"; then
-        CFLAGS_TUNING="-march=core2 -mtune=core2"
-    elif test "${host_cpu}" = "i686" -o "${host_cpu}" = "i586" -o "${host_cpu}" = "i486" -o "${host_cpu}" = "i386"; then
-        CFLAGS_TUNING="-mtune=pentium2"
-    elif test "${host_cpu}" = "x86_64"; then
-        CFLAGS_TUNING="-mtune=athlon64"
-    elif test "${host_cpu}" = "powerpc"; then
-        CFLAGS_TUNING="-mtune=G4";
-    fi
-fi
-
-dnl NOTE: this can't be cached cleanly
-AS_IF([test -n "${CFLAGS_TUNING}"], [
-  VLC_SAVE_FLAGS
-  CFLAGS="${CFLAGS} ${CFLAGS_TUNING}"
-  AC_MSG_CHECKING([whether $CC accepts ${CFLAGS_TUNING}])
-  AC_COMPILE_IFELSE([AC_LANG_SOURCE([])], [
-    AC_MSG_RESULT([yes])
-  ], [
-    VLC_RESTORE_FLAGS
-    AC_MSG_RESULT([no])
-    AS_IF([test "${with_tuning}"], [AC_MSG_ERROR([requested tuning not supported])])
-  ])
-])
-
 dnl
 dnl  Memory usage
 dnl