Reorder the if/else for residual cats to match gcc's branch prediction.
authorAlexander Strange <astrange@ithinksw.com>
Wed, 2 Jul 2008 22:20:06 +0000 (22:20 +0000)
committerAlexander Strange <astrange@ithinksw.com>
Wed, 2 Jul 2008 22:20:06 +0000 (22:20 +0000)
Originally committed as revision 14053 to svn://svn.ffmpeg.org/ffmpeg/trunk

libavcodec/h264.c

index 57e97e3..561d68e 100644 (file)
@@ -5350,13 +5350,13 @@ static av_always_inline int get_cabac_cbf_ctx( H264Context *h, int cat, int idx,
             nzb = (h-> top_cbp>>(6+idx))&0x01;
         }
     } else {
-        if( cat == 1 || cat == 2 ) {
-            nza = h->non_zero_count_cache[scan8[idx] - 1];
-            nzb = h->non_zero_count_cache[scan8[idx] - 8];
-        } else {
-            assert(cat == 4);
+        if( cat == 4 ) {
             nza = h->non_zero_count_cache[scan8[16+idx] - 1];
             nzb = h->non_zero_count_cache[scan8[16+idx] - 8];
+        } else {
+            assert(cat == 1 || cat == 2);
+            nza = h->non_zero_count_cache[scan8[idx] - 1];
+            nzb = h->non_zero_count_cache[scan8[idx] - 8];
         }
     }
 
@@ -5447,10 +5447,10 @@ static av_always_inline void decode_cabac_residual_internal( H264Context *h, DCT
     if( is_dc || cat != 5 ) {
         if( get_cabac( CC, &h->cabac_state[85 + get_cabac_cbf_ctx( h, cat, n, is_dc ) ] ) == 0 ) {
             if( !is_dc ) {
-                if( cat == 1 || cat == 2 )
-                    h->non_zero_count_cache[scan8[n]] = 0;
-                else
+                if( cat == 4 )
                     h->non_zero_count_cache[scan8[16+n]] = 0;
+                else
+                    h->non_zero_count_cache[scan8[n]] = 0;
             }
 
 #ifdef CABAC_ON_STACK
@@ -5504,13 +5504,13 @@ static av_always_inline void decode_cabac_residual_internal( H264Context *h, DCT
         else
             h->cbp_table[h->mb_xy] |= 0x40 << n;
     } else {
-        if( cat == 1 || cat == 2 )
-            h->non_zero_count_cache[scan8[n]] = coeff_count;
+        if( cat == 5 )
+            fill_rectangle(&h->non_zero_count_cache[scan8[n]], 2, 2, 8, coeff_count, 1);
         else if( cat == 4 )
             h->non_zero_count_cache[scan8[16+n]] = coeff_count;
         else {
-            assert( cat == 5 );
-            fill_rectangle(&h->non_zero_count_cache[scan8[n]], 2, 2, 8, coeff_count, 1);
+            assert( cat == 1 || cat == 2 );
+            h->non_zero_count_cache[scan8[n]] = coeff_count;
         }
     }