Jean-Baptiste Kempf <[email protected]> writes:

> ---
>  libavcodec/arm/rv34dsp_init_neon.c | 5 +++++
>  libavcodec/arm/rv40dsp_init_neon.c | 5 +++++
>  2 files changed, 10 insertions(+)
>
> diff --git a/libavcodec/arm/rv34dsp_init_neon.c 
> b/libavcodec/arm/rv34dsp_init_neon.c
> index b4a2f01..3abc2c6 100644
> --- a/libavcodec/arm/rv34dsp_init_neon.c
> +++ b/libavcodec/arm/rv34dsp_init_neon.c
> @@ -22,6 +22,7 @@
>
>  #include "libavcodec/avcodec.h"
>  #include "libavcodec/rv34dsp.h"
> +#include "libavutil/arm/cpu.h"
>
>  void ff_rv34_inv_transform_noround_neon(DCTELEM *block);
>
> @@ -32,9 +33,13 @@ void ff_rv34_idct_dc_add_neon(uint8_t *dst, ptrdiff_t 
> stride, int dc);
>
>  void ff_rv34dsp_init_neon(RV34DSPContext *c, DSPContext* dsp)
>  {
> +    int cpu_flags = av_get_cpu_flags();
> +
> +    if (have_neon(cpu_flags)) {

Wrong.  That function shouldn't be called at all if NEON isn't enabled.
The rv*dsp init stuff looks weird.  I guess I'll have to fix it myself.

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

Reply via email to