Commit 828cfff6 authored by Michael Niedermayer's avatar Michael Niedermayer

Pack MVs together from the begin for spatial direct, this simplifies the code

and is a bit faster (5-10 cpu cycles depending on what is meassured).

Originally committed as revision 21784 to svn://svn.ffmpeg.org/ffmpeg/trunk
parent 02e8c5f0
...@@ -216,7 +216,7 @@ single_col: ...@@ -216,7 +216,7 @@ single_col:
if(h->direct_spatial_mv_pred){ if(h->direct_spatial_mv_pred){
int ref[2]; int ref[2];
int mv[2][2]; int mv[2];
int list; int list;
/* ref = min(neighbors) */ /* ref = min(neighbors) */
...@@ -237,24 +237,21 @@ single_col: ...@@ -237,24 +237,21 @@ single_col:
int match_count= (left_ref==ref[list]) + (top_ref==ref[list]) + (refc==ref[list]); int match_count= (left_ref==ref[list]) + (top_ref==ref[list]) + (refc==ref[list]);
if(match_count > 1){ //most common if(match_count > 1){ //most common
mv[list][0]= mid_pred(A[0], B[0], C[0]); mv[list]= (mid_pred(A[0], B[0], C[0])&0xFFFF)
mv[list][1]= mid_pred(A[1], B[1], C[1]); +(mid_pred(A[1], B[1], C[1])<<16);
}else { }else {
assert(match_count==1); assert(match_count==1);
if(left_ref==ref[list]){ if(left_ref==ref[list]){
mv[list][0]= A[0]; mv[list]= *(uint32_t*)A;
mv[list][1]= A[1];
}else if(top_ref==ref[list]){ }else if(top_ref==ref[list]){
mv[list][0]= B[0]; mv[list]= *(uint32_t*)B;
mv[list][1]= B[1];
}else{ }else{
mv[list][0]= C[0]; mv[list]= *(uint32_t*)C;
mv[list][1]= C[1];
} }
} }
}else{ }else{
int mask= ~(MB_TYPE_L0 << (2*list)); int mask= ~(MB_TYPE_L0 << (2*list));
mv[list][0] = mv[list][1] = 0; mv[list] = 0;
ref[list] = -1; ref[list] = -1;
if(!is_b8x8) if(!is_b8x8)
*mb_type &= mask; *mb_type &= mask;
...@@ -288,13 +285,13 @@ single_col: ...@@ -288,13 +285,13 @@ single_col:
|| (l1ref0[xy8] < 0 && l1ref1[xy8] == 0 && FFABS(l1mv1[xy4][0]) <= 1 && FFABS(l1mv1[xy4][1]) <= 1))){ || (l1ref0[xy8] < 0 && l1ref1[xy8] == 0 && FFABS(l1mv1[xy4][0]) <= 1 && FFABS(l1mv1[xy4][1]) <= 1))){
a=b=0; a=b=0;
if(ref[0] > 0) if(ref[0] > 0)
a= pack16to32(mv[0][0],mv[0][1]); a= mv[0];
if(ref[1] > 0) if(ref[1] > 0)
b= pack16to32(mv[1][0],mv[1][1]); b= mv[1];
n++; n++;
}else{ }else{
a= pack16to32(mv[0][0],mv[0][1]); a= mv[0];
b= pack16to32(mv[1][0],mv[1][1]); b= mv[1];
} }
fill_rectangle(&h->mv_cache[0][scan8[i8*4]], 2, 2, 8, a, 4); fill_rectangle(&h->mv_cache[0][scan8[i8*4]], 2, 2, 8, a, 4);
fill_rectangle(&h->mv_cache[1][scan8[i8*4]], 2, 2, 8, b, 4); fill_rectangle(&h->mv_cache[1][scan8[i8*4]], 2, 2, 8, b, 4);
...@@ -312,12 +309,12 @@ single_col: ...@@ -312,12 +309,12 @@ single_col:
&& h->x264_build>33U))){ && h->x264_build>33U))){
a=b=0; a=b=0;
if(ref[0] > 0) if(ref[0] > 0)
a= pack16to32(mv[0][0],mv[0][1]); a= mv[0];
if(ref[1] > 0) if(ref[1] > 0)
b= pack16to32(mv[1][0],mv[1][1]); b= mv[1];
}else{ }else{
a= pack16to32(mv[0][0],mv[0][1]); a= mv[0];
b= pack16to32(mv[1][0],mv[1][1]); b= mv[1];
} }
fill_rectangle(&h->mv_cache[0][scan8[0]], 4, 4, 8, a, 4); fill_rectangle(&h->mv_cache[0][scan8[0]], 4, 4, 8, a, 4);
fill_rectangle(&h->mv_cache[1][scan8[0]], 4, 4, 8, b, 4); fill_rectangle(&h->mv_cache[1][scan8[0]], 4, 4, 8, b, 4);
...@@ -331,8 +328,8 @@ single_col: ...@@ -331,8 +328,8 @@ single_col:
continue; continue;
h->sub_mb_type[i8] = sub_mb_type; h->sub_mb_type[i8] = sub_mb_type;
fill_rectangle(&h->mv_cache[0][scan8[i8*4]], 2, 2, 8, pack16to32(mv[0][0],mv[0][1]), 4); fill_rectangle(&h->mv_cache[0][scan8[i8*4]], 2, 2, 8, mv[0], 4);
fill_rectangle(&h->mv_cache[1][scan8[i8*4]], 2, 2, 8, pack16to32(mv[1][0],mv[1][1]), 4); fill_rectangle(&h->mv_cache[1][scan8[i8*4]], 2, 2, 8, mv[1], 4);
fill_rectangle(&h->ref_cache[0][scan8[i8*4]], 2, 2, 8, (uint8_t)ref[0], 1); fill_rectangle(&h->ref_cache[0][scan8[i8*4]], 2, 2, 8, (uint8_t)ref[0], 1);
fill_rectangle(&h->ref_cache[1][scan8[i8*4]], 2, 2, 8, (uint8_t)ref[1], 1); fill_rectangle(&h->ref_cache[1][scan8[i8*4]], 2, 2, 8, (uint8_t)ref[1], 1);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment