[Ffmpeg-devel] PATCH Blackfin optimized byte swapping mechanism

[Michael Niedermayer]

Hi

On Mon, Apr 23, 2007 at 07:46:40AM -0400, Marc Hoffman wrote:
Content-Description: message body text
> Marc Hoffman writes:
>  > Michael Niedermayer writes:
>  >  > Hi
>  >  > 
>  >  > On Tue, Apr 17, 2007 at 07:40:47AM -0400, Marc Hoffman wrote:
>  >  > Content-Description: message body text
>  >  > > 
>  >  > >  > Low level bswap primitive for the Blackfin Architecture.
>  >  > > 
>  >  > > sorry mangled patch wrong encoding last time.
>  >  > 
>  >  > what advantage do these functions have over the default?
>  >  > are they faster? if so you should provide some benchmarks
>  > 
>  > Sorry about the top post please forgive me
>  > 
>  > So I guess this is about 300% improvement in performance for this function.
>  > 
>  > Marc
>  > 
> 
> I think this got lost. Can this be reviewed thanks.
> 

> Index: bswap.h
> ===================================================================
> --- bswap.h	(revision 8596)
> +++ bswap.h	(working copy)
> @@ -108,6 +108,36 @@
>      r.l.h = bswap_32 (x>>32);
>      return r.ll;
>  }
> +#elif defined(ARCH_BFIN)
> +
> +static av_always_inline uint16_t bswap_16(uint16_t x){
> +    return (x>>8) | (x<<8);
> +}

code duplication


[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Breaking DRM is a little like attempting to break through a door even
though the window is wide open and the only thing in the house is a bunch
of things you dont want and which you would get tomorrow for free anyway
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070423/cb329876/attachment.pgp>