[FFmpeg-devel] [PATCH] h264_intrapred: Remove ff_pred16x16_horizontal_8_mmxext

[Kieran Kunhya]

On Sun, 11 Feb 2024 at 22:33, Andreas Rheinhardt <
andreas.rheinhardt at outlook.com> wrote:

> James Almer:
> > On 2/11/2024 6:24 PM, Kieran Kunhya wrote:
> >> $subj, now with forward declaration also removed.
> >
> > This function is trivial to convert to SSE2, so better do that than
> > removing it. Attached.
> > If other functions are harder to port to SSE2, then sure, they can be
> > removed.
> >
>
> Benchmarks?
>
> - Andreas
>

For me on Haswell x64:

pred16x16_horizontal_8_c: 41.5
pred16x16_horizontal_8_sse2: 32.5
pred16x16_horizontal_8_ssse3: 9.0

pred16x16_horizontal_8_c: 43.5
pred16x16_horizontal_8_mmxext: 20.7
pred16x16_horizontal_8_ssse3: 12.2

Kieran