[Ffmpeg-devel] [PATCH] dead code elimination

Diego Biurrun diego
Mon Feb 26 11:47:45 CET 2007


On Mon, Feb 26, 2007 at 02:13:09AM +0100, Aurelien Jacobs wrote:
> On Mon, 26 Feb 2007 00:46:38 +0000
> M?ns Rullg?rd <mans at mansr.com> wrote:
> 
> > Not OK.  This should be set for all 64-bit CPUs we currently have
> > special settings for.
> 
> OK. I first wanted to be on a safe side, but indeed, if it's not changed right
> now for all 64 bits arch, I may never be changed.
> 
> > These include Alpha, PPC64, and Sparc64.
> 
> I guess this also includes ia64 and parisc64 ;-)
> 
> Updated patch attached.

> --- configure	(revision 8129)
> +++ configure	(working copy)
> @@ -966,15 +968,21 @@
>    ;;
>    alpha)
>      arch="alpha"
> +    enable fast_64bit
>    ;;
> -  "Power Macintosh"|ppc|ppc64|powerpc)
> +  "Power Macintosh"|ppc|powerpc)
>      arch="powerpc"
>    ;;
> +  ppc64)
> +    arch="powerpc"
> +    enable fast_64bit
> +  ;;
>    mips|mipsel|IP*)
>      arch="mips"
>    ;;
>    sun4u|sparc64)
>      arch="sparc64"
> +    enable fast_64bit
>    ;;
>    sparc)
>      arch="sparc"
> @@ -982,9 +990,13 @@
>    sh4)
>      arch="sh4"
>    ;;
> -  parisc|parisc64)
> +  parisc)
>      arch="parisc"
>    ;;
> +  parisc64)
> +    arch="parisc"
> +    enable fast_64bit
> +  ;;
>    s390|s390x)
>      arch="s390"
>    ;;

The partial duplication is somewhat ugly, but I don't see a way around
it.  Patch OK with me.

Diego




More information about the ffmpeg-devel mailing list