[x264-devel] [PATCH] Delete all SPARC optimizations

Janne Grunau janne-x264 at jannau.net
Sun May 4 10:24:36 CEST 2014


On 2014-04-27 22:56:34 +0200, Diego Biurrun wrote:
> SPARC has been obsolete for a long time and makes little sense as a
> H.264 encoding platform.
> ---
> 
> Saving 1150 lines is not too shabby and SPARC went the way of the dodo
> a long time ago ...
> 
>  Makefile               |    8 -
>  common/pixel.c         |   19 -
>  common/sparc/pixel.asm | 1089 ------------------------------------------------
>  common/sparc/pixel.h   |   34 --
>  configure              |    3 +-
>  5 files changed, 1 insertion(+), 1152 deletions(-)
>  delete mode 100644 common/sparc/pixel.asm
>  delete mode 100644 common/sparc/pixel.h

ok, except configure

> diff --git a/configure b/configure
> index 5ef4103..4a2e73a 100755
> --- a/configure
> +++ b/configure
> @@ -636,7 +636,6 @@ case $host_cpu in
>          case $(uname -m) in
>              sun4u|sun4v)
>                  if [ $asm = auto ]; then
> -                    ARCH="UltraSPARC"
>                      if ! echo $CFLAGS | grep -Eq '\-mcpu' ; then
>                          CFLAGS="$CFLAGS -mcpu=ultrasparc"
>                          LDFLAGS="$LDFLAGS -mcpu=ultrasparc"

I think there is little sense in keeping the case $(uname -m) block.
Adding -mcpu=ultrasparc was done to ensure that the asm compiles.

Janne


More information about the x264-devel mailing list