From c212fb0cb1900ab92c00de948fb25f00c93639bb Mon Sep 17 00:00:00 2001 From: Michael Niedermayer Date: Sun, 21 Dec 2008 15:58:42 +0000 Subject: [PATCH] Only execute clear_blocks() when needed. +0.3% speedup for both aladin & cathedral. Originally committed as revision 16252 to svn://svn.ffmpeg.org/ffmpeg/trunk --- libavcodec/h264.c | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/libavcodec/h264.c b/libavcodec/h264.c index 202ff91f92..2c065c79de 100644 --- a/libavcodec/h264.c +++ b/libavcodec/h264.c @@ -2595,6 +2595,9 @@ static av_always_inline void hl_decode_mb_internal(H264Context *h, int simple){ } } } + if(h->cbp || IS_INTRA(mb_type)) + s->dsp.clear_blocks(h->mb); + if(h->deblocking_filter) { backup_mb_border(h, dest_y, dest_cb, dest_cr, linesize, uvlinesize, simple); fill_caches(h, mb_type, 1); //FIXME don't fill stuff which isn't used by filter_mb @@ -4262,8 +4265,6 @@ static int decode_mb_cavlc(H264Context *h){ mb_xy = h->mb_xy = s->mb_x + s->mb_y*s->mb_stride; - s->dsp.clear_blocks(h->mb); //FIXME avoid if already clear (move after skip handlong? - tprintf(s->avctx, "pic:%d mb:%d/%d\n", h->frame_num, s->mb_x, s->mb_y); cbp = 0; /* avoid warning. FIXME: find a solution without slowing down the code */ @@ -5340,8 +5341,6 @@ static int decode_mb_cabac(H264Context *h) { mb_xy = h->mb_xy = s->mb_x + s->mb_y*s->mb_stride; - s->dsp.clear_blocks(h->mb); //FIXME avoid if already clear (move after skip handlong?) - tprintf(s->avctx, "pic:%d mb:%d/%d\n", h->frame_num, s->mb_x, s->mb_y); if( h->slice_type_nos != FF_I_TYPE ) { int skip;