[FFmpeg-cvslog] r9307 - in trunk/libavcodec: dsputil.c mpegvideo.c

[Reimar Doeffinger]

Hello,
On Thu, Jun 14, 2007 at 03:20:24PM +0300, Jan Knutar wrote:
> On Thursday 14 June 2007 12:06, Reimar Doeffinger wrote:
> > > -#ifdef HAVE_MLIB
> > > +#elif defined(HAVE_MLIB)
> > >      dsputil_init_mlib(c, avctx);
> > > -#endif
> > > -#ifdef ARCH_SPARC
> > > +#elif defined(ARCH_SPARC)
> > >     dsputil_init_vis(c,avctx);
> > > -#endif
> >
> > Are you sure about that? I would have thought this was on purpose,
> > because it would allow (as a made-up example) to use full
> > optimizations if only some dsp functions are optimized via SPARC asm
> > and some via MLIB.
> 
> Sparc asm should probably be preferred over mlib. iirc delewis said the 
> mlib stuff performs quite horribly, if it even worked at all..

Well, in that case this patch even broke things.

Greetings,
Reimar Doeffinger