[FFmpeg-devel] [PATCH] Port MPlayer 2xSaI filter to libavfilter

[Nielkie]

On Mon, Nov 29, 2010 at 12:13 AM, Ronald S. Bultje <rsbultje at gmail.com>wrote:

> Hi,
>
> On Sun, Nov 28, 2010 at 2:54 AM, Nielkie <nielkie at gmail.com> wrote:
> > On Sat, Nov 27, 2010 at 8:29 PM, Reimar D?ffinger
> > <Reimar.Doeffinger at gmx.de>wrote:
> >> On Sat, Nov 27, 2010 at 07:52:52PM +1000, Nielkie wrote:
> >> > +            /* Move color matrix forward */
> >> > +            for (my = 0; my < 4; my++) {
> >> > +                color[my][0] = color[my][1];
> >> > +                color[my][1] = color[my][2];
> >> > +                color[my][2] = color[my][3];
> >> > +            }
> >>
> >> Using a loop here seems likely to have a major negative impact on
> >> performance.
> >
> > I'm not sure, I would think the compiler would unroll it.
>
> Could you verify that please?
>
> Ronald
>
>
I can confirm that it is unrolled on gcc 4.2.1, at least.