[PATCH] New rgb32tobgr32 (was: Re: [Ffmpeg-devel] [PATCH] have cs_test check for sigsegv at smaller widths and sigill)

[Ivo]

On Saturday 14 April 2007 17:32, Michael Niedermayer wrote:
> On Sat, Apr 14, 2007 at 02:27:35PM +0200, Ivo wrote:
> > > > +		"	movq %%mm0, %%mm2		\n"
> > > > +		"	movq %%mm1, %%mm4		\n"
> > > > +		"	pand %%mm7, %%mm0		\n"
> > > > +		"	pand %%mm6, %%mm2		\n"
> > > > +		"	pand %%mm7, %%mm1		\n"
> > > > +		"	pand %%mm6, %%mm4		\n"
> > > > +# ifdef HAVE_MMX2
> > > > +		"	pshufw $177, %%mm2, %%mm3	\n"
> > > > +		"	pshufw $177, %%mm4, %%mm5	\n"
> > > > +		"	por %%mm3, %%mm0		\n"
> > > > +		"	por %%mm5, %%mm1		\n"
> > >
> > > you can still avoid 2 movq here, that is
> > > read X
> > > pshufw X,Y
> > > pand C0,X
> > > pand C1,Y
> > > por  X,Y
> > > store Y
> >
> > Done.
>
> looks ok

Applied.

--Ivo