Luca Barbato <[email protected]> writes: > 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?
Yes, that is assumed. I don't currently see any point in doing anything more elaborate. -- Måns Rullgård [email protected] _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
