[MPlayer-dev-eng] [PATCH] Suppress unsupported options when building with clang

[Diego Biurrun]

.. extra good karma for cutting down on line length ..

On Mon, Jul 26, 2010 at 06:49:45PM +1000, Rowan James wrote:
> 
> --- configure	(revision 31826)
> +++ configure	(working copy)
> @@ -1505,7 +1505,10 @@
>  
>  if darwin; then
> -  extra_cflags="-mdynamic-no-pic -falign-loops=16 -shared-libgcc $extra_cflags"
> +  extra_cflags="-mdynamic-no-pic $extra_cflags"
> +  if test "$(basename $_cc)" != "clang" ; then
> +    extra_cflags="-falign-loops=16 -shared-libgcc $extra_cflags"
> +  fi

I haven't investigated in detail yet, but this does not look like the
default way to identify gcc.

> @@ -2539,6 +2542,8 @@
>    elif test "$cc_vendor" = "sun" ; then
>      CFLAGS="-O2 $_march $_mcpu $_pipe -xc99 -xregs=frameptr"
> +  elif test "$cc_vendor" = "clang"; then
> +    CFLAGS="-O2 $_march $_pipe"
>    elif test "$cc_vendor" != "gnu" ; then
>      CFLAGS="-O2 $_march $_mcpu $_pipe"

This hunk is good anyway, applied.

Diego