[Ffmpeg-devel] [PATCH] fix mpegaudiodec on ARM and benchmark

[Diego Biurrun]

On Wed, Sep 13, 2006 at 10:38:40PM +0300, Siarhei Siamashka wrote:
> On Wednesday 13 September 2006 12:27, Diego Biurrun wrote:
> 
> > On Wed, Sep 13, 2006 at 10:52:07AM +0200, Guillaume POIRIER wrote:
> > > On 9/12/06, Siarhei Siamashka <siarhei.siamashka at gmail.com> wrote:
> > > >Also it is not quite related to ffmpeg, but mplayer can't be
> > > >configured to use HAVE_IWMMXT, so it does not use full optimizations
> > > >on xscale cpu such as yours, maybe it is a good idea to fix it?
> > >
> > > Yes, sure. I'm quite confident that all it takes it to add a configure
> > > check.
> >
> > Yes, patch welcome.
> 
> In the mean time here is a patch for ffmpeg configure script adding 
> armv5te instructions support detection (pretty much a copy-paste from
> iwmmxt detection). Also changed mpegaudiodec.c to use this new 
> HAVE_ARMV5TE define.
> 
> --- configure	(revision 6244)
> +++ configure	(working copy)
> @@ -1526,6 +1540,9 @@
>      echo "3DNow! Builtins  $mm3dnow"
>  fi
>  if test $cpu = "armv4l"; then
> +    echo "ARMv5TE enabled  $armv5te"
> +fi
> +if test $cpu = "armv4l"; then
>      echo "IWMMXT enabled   $iwmmxt"
>  fi

No need for the extra if check here, it's identical to the one above.

Patch applied with that small modification.

Diego