[MPlayer-dev-eng] [PATCH] mp3lib: Align output so that movaps can be used instead of movups

Guillaume POIRIER poirierg at gmail.com
Mon Jun 4 08:47:10 CEST 2007


Hi,

On 6/4/07, Zuxy Meng <zuxy.meng at gmail.com> wrote:

> The attached patch aligns output address of dct64 functions to 16-byte
> boundary, so movaps can be used instead of movups in dct64_sse,
> resulting in 2.5% faster decode.

I think it's better to use DECLARE_ALIGNED_16() macros. If they aren't
available in mp3lib, then I think it's OK to add #include the relevant
file from lavc.

If someone disagrees with this, please speak up!


> I tested against mp3 and mp2 and both worked fine; couldn't find an
> mp1 sample though.

I think that should be one:
http://samples.mplayerhq.hu/A-codecs/MP3/mpeg_layer1_audio.mpg

Guillaume
-- 
Y'a pas de gonzesse hooligan,
Imbécile et meurtrière
Y'en a pas même en grande Bretagne
A part bien sûr Madame Thatcher
  -- Renaud (sur "Miss Maggie")


More information about the MPlayer-dev-eng mailing list