[FFmpeg-devel] AVX assembly for ffmpeg

[Reinhard Tartler]

On Sun, Feb 20, 2011 at 01:37:31 (CET), Ronald S. Bultje wrote:

> Hi,
>
> On Sat, Feb 19, 2011 at 2:16 PM, Daniel Kang <daniel.d.kang at gmail.com> wrote:
>> I don't have a Sandy Bridge to test this for speed, but it compiles and
>> passes make fate-h264. More development will probably have to wait until
>> March, when Jason has access to a Sandy Bridge (and can give me access
>> to it).
> [..]
>> +#if HAVE_AVX
>> +#if CONFIG_GPL
>> +    if (mm_flags & AV_CPU_FLAG_AVX) {
>> +        h->pred8x8l [DIAG_DOWN_RIGHT_PRED] = ff_pred8x8l_down_right_avx;
>> +    }
>> +#endif
>> +#endif
>
> #if HAVE_AVX && CONFIG_GPL
>
> Other than that, let's make sure this stuff speeds it up, otherwise
> it's pointless. Once you have Intel's Sandy Bridge and can test it,
> let's make sure it works and then it's fine to commit with me.

I tested it on my sandy bridge and can confirm that this patch works.

it doesn't bring performance yet, but as jason points out, this patch is
about introducing infrastructure, and this works.

-- 
Gruesse/greetings,
Reinhard Tartler, KeyID 945348A4