On 06/23/2012 03:25 PM, Mans Rullgard wrote:
> This removes a dependency on implementation details from generic
> code and allows easy addition of the equivalent optimisation for
> other architectures than x86.
> 
> Signed-off-by: Mans Rullgard <[email protected]>
> ---
>  libavcodec/h264_cabac.c    |    6 +++---
>  libavcodec/x86/h264_i386.h |    2 ++
>  2 files changed, 5 insertions(+), 3 deletions(-)
> 
> diff --git a/libavcodec/h264_cabac.c b/libavcodec/h264_cabac.c
> index 08a6a5b..2bf08b5 100644
> --- a/libavcodec/h264_cabac.c
> +++ b/libavcodec/h264_cabac.c
> @@ -1652,14 +1652,14 @@ decode_cabac_residual_internal(H264Context *h, 
> DCTELEM *block,
>              index[coeff_count++] = last;\
>          }
>          const uint8_t *sig_off = significant_coeff_flag_offset_8x8[MB_FIELD];
> -#if ARCH_X86 && HAVE_7REGS
> -        coeff_count= decode_significance_8x8_x86(CC, 
> significant_coeff_ctx_base, index,
> +#ifdef decode_significance

decode_significance and decode_significance_8x8 must always exist
together, isn't it?

The rest seems fine.

lu

-- 

Luca Barbato
Gentoo/linux
http://dev.gentoo.org/~lu_zero

_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to