"Ronald S. Bultje" <[email protected]> writes:

> From: "Ronald S. Bultje" <[email protected]>
>
> ---
>  libavcodec/x86/vp8dsp-init.c |    6 ++++++
>  1 file changed, 6 insertions(+)
>
> diff --git a/libavcodec/x86/vp8dsp-init.c b/libavcodec/x86/vp8dsp-init.c
> index d3f1456..a0e8f9b 100644
> --- a/libavcodec/x86/vp8dsp-init.c
> +++ b/libavcodec/x86/vp8dsp-init.c
> @@ -401,11 +401,13 @@ av_cold void ff_vp8dsp_init_x86(VP8DSPContext* c)
>
>          c->vp8_h_loop_filter_simple = ff_vp8_h_loop_filter_simple_sse2;
>
> +#if ARCH_X86_64 || HAVE_ALIGNED_STACK

ARCH_X86_64 sets HAVE_ALIGNED_STACK.

>          c->vp8_h_loop_filter16y_inner = ff_vp8_h_loop_filter16y_inner_sse2;
>          c->vp8_h_loop_filter8uv_inner = ff_vp8_h_loop_filter8uv_inner_sse2;
>
>          c->vp8_h_loop_filter16y       = ff_vp8_h_loop_filter16y_mbedge_sse2;
>          c->vp8_h_loop_filter8uv       = ff_vp8_h_loop_filter8uv_mbedge_sse2;
> +#endif

Wouldn't it be better to make these functions align the stack manually?
Not having these optimisations seems like a rather bad thing.

-- 
Måns Rullgård
[email protected]
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to