Merge "Moving {sb, mb, b, ab}_index from MACROBLOCKD to MACROBLOCK."

This commit is contained in:
Dmitry Kovalev
2013-11-12 15:59:28 -08:00
committed by Gerrit Code Review
5 changed files with 70 additions and 82 deletions

View File

@@ -215,11 +215,6 @@ typedef struct macroblockd {
int corrupted; int corrupted;
unsigned char sb_index; // index of 32x32 block inside the 64x64 block
unsigned char mb_index; // index of 16x16 block inside the 32x32 block
unsigned char b_index; // index of 8x8 block inside the 16x16 block
unsigned char ab_index; // index of 4x4 block inside the 8x8 block
int q_index; int q_index;
/* Y,U,V,(A) */ /* Y,U,V,(A) */

View File

@@ -131,6 +131,11 @@ struct macroblock {
int y_mode_costs[INTRA_MODES][INTRA_MODES][INTRA_MODES]; int y_mode_costs[INTRA_MODES][INTRA_MODES][INTRA_MODES];
int switchable_interp_costs[SWITCHABLE_FILTER_CONTEXTS][SWITCHABLE_FILTERS]; int switchable_interp_costs[SWITCHABLE_FILTER_CONTEXTS][SWITCHABLE_FILTERS];
unsigned char sb_index; // index of 32x32 block inside the 64x64 block
unsigned char mb_index; // index of 16x16 block inside the 32x32 block
unsigned char b_index; // index of 8x8 block inside the 16x16 block
unsigned char ab_index; // index of 4x4 block inside the 8x8 block
// These define limits to motion vector components to prevent them // These define limits to motion vector components to prevent them
// from extending outside the UMV borders // from extending outside the UMV borders
int mv_col_min; int mv_col_min;
@@ -193,35 +198,33 @@ struct macroblock {
// refactoring on organizing the temporary buffers, when recursive // refactoring on organizing the temporary buffers, when recursive
// partition down to 4x4 block size is enabled. // partition down to 4x4 block size is enabled.
static PICK_MODE_CONTEXT *get_block_context(MACROBLOCK *x, BLOCK_SIZE bsize) { static PICK_MODE_CONTEXT *get_block_context(MACROBLOCK *x, BLOCK_SIZE bsize) {
MACROBLOCKD *const xd = &x->e_mbd;
switch (bsize) { switch (bsize) {
case BLOCK_64X64: case BLOCK_64X64:
return &x->sb64_context; return &x->sb64_context;
case BLOCK_64X32: case BLOCK_64X32:
return &x->sb64x32_context[xd->sb_index]; return &x->sb64x32_context[x->sb_index];
case BLOCK_32X64: case BLOCK_32X64:
return &x->sb32x64_context[xd->sb_index]; return &x->sb32x64_context[x->sb_index];
case BLOCK_32X32: case BLOCK_32X32:
return &x->sb32_context[xd->sb_index]; return &x->sb32_context[x->sb_index];
case BLOCK_32X16: case BLOCK_32X16:
return &x->sb32x16_context[xd->sb_index][xd->mb_index]; return &x->sb32x16_context[x->sb_index][x->mb_index];
case BLOCK_16X32: case BLOCK_16X32:
return &x->sb16x32_context[xd->sb_index][xd->mb_index]; return &x->sb16x32_context[x->sb_index][x->mb_index];
case BLOCK_16X16: case BLOCK_16X16:
return &x->mb_context[xd->sb_index][xd->mb_index]; return &x->mb_context[x->sb_index][x->mb_index];
case BLOCK_16X8: case BLOCK_16X8:
return &x->sb16x8_context[xd->sb_index][xd->mb_index][xd->b_index]; return &x->sb16x8_context[x->sb_index][x->mb_index][x->b_index];
case BLOCK_8X16: case BLOCK_8X16:
return &x->sb8x16_context[xd->sb_index][xd->mb_index][xd->b_index]; return &x->sb8x16_context[x->sb_index][x->mb_index][x->b_index];
case BLOCK_8X8: case BLOCK_8X8:
return &x->sb8x8_context[xd->sb_index][xd->mb_index][xd->b_index]; return &x->sb8x8_context[x->sb_index][x->mb_index][x->b_index];
case BLOCK_8X4: case BLOCK_8X4:
return &x->sb8x4_context[xd->sb_index][xd->mb_index][xd->b_index]; return &x->sb8x4_context[x->sb_index][x->mb_index][x->b_index];
case BLOCK_4X8: case BLOCK_4X8:
return &x->sb4x8_context[xd->sb_index][xd->mb_index][xd->b_index]; return &x->sb4x8_context[x->sb_index][x->mb_index][x->b_index];
case BLOCK_4X4: case BLOCK_4X4:
return &x->ab4x4_context[xd->sb_index][xd->mb_index][xd->b_index]; return &x->ab4x4_context[x->sb_index][x->mb_index][x->b_index];
default: default:
assert(0); assert(0);
return NULL; return NULL;

View File

@@ -50,25 +50,25 @@
int enc_debug = 0; int enc_debug = 0;
#endif #endif
static INLINE uint8_t *get_sb_index(MACROBLOCKD *xd, BLOCK_SIZE subsize) { static INLINE uint8_t *get_sb_index(MACROBLOCK *x, BLOCK_SIZE subsize) {
switch (subsize) { switch (subsize) {
case BLOCK_64X64: case BLOCK_64X64:
case BLOCK_64X32: case BLOCK_64X32:
case BLOCK_32X64: case BLOCK_32X64:
case BLOCK_32X32: case BLOCK_32X32:
return &xd->sb_index; return &x->sb_index;
case BLOCK_32X16: case BLOCK_32X16:
case BLOCK_16X32: case BLOCK_16X32:
case BLOCK_16X16: case BLOCK_16X16:
return &xd->mb_index; return &x->mb_index;
case BLOCK_16X8: case BLOCK_16X8:
case BLOCK_8X16: case BLOCK_8X16:
case BLOCK_8X8: case BLOCK_8X8:
return &xd->b_index; return &x->b_index;
case BLOCK_8X4: case BLOCK_8X4:
case BLOCK_4X8: case BLOCK_4X8:
case BLOCK_4X4: case BLOCK_4X4:
return &xd->ab_index; return &x->ab_index;
default: default:
assert(0); assert(0);
return NULL; return NULL;
@@ -602,7 +602,7 @@ static void pick_sb_modes(VP9_COMP *cpi, const TileInfo *const tile,
if (bsize < BLOCK_8X8) { if (bsize < BLOCK_8X8) {
// When ab_index = 0 all sub-blocks are handled, so for ab_index != 0 // When ab_index = 0 all sub-blocks are handled, so for ab_index != 0
// there is nothing to be done. // there is nothing to be done.
if (xd->ab_index != 0) { if (x->ab_index != 0) {
*totalrate = 0; *totalrate = 0;
*totaldist = 0; *totaldist = 0;
return; return;
@@ -706,16 +706,15 @@ static void update_stats(VP9_COMP *cpi) {
} }
static BLOCK_SIZE *get_sb_partitioning(MACROBLOCK *x, BLOCK_SIZE bsize) { static BLOCK_SIZE *get_sb_partitioning(MACROBLOCK *x, BLOCK_SIZE bsize) {
MACROBLOCKD *const xd = &x->e_mbd;
switch (bsize) { switch (bsize) {
case BLOCK_64X64: case BLOCK_64X64:
return &x->sb64_partitioning; return &x->sb64_partitioning;
case BLOCK_32X32: case BLOCK_32X32:
return &x->sb_partitioning[xd->sb_index]; return &x->sb_partitioning[x->sb_index];
case BLOCK_16X16: case BLOCK_16X16:
return &x->mb_partitioning[xd->sb_index][xd->mb_index]; return &x->mb_partitioning[x->sb_index][x->mb_index];
case BLOCK_8X8: case BLOCK_8X8:
return &x->b_partitioning[xd->sb_index][xd->mb_index][xd->b_index]; return &x->b_partitioning[x->sb_index][x->mb_index][x->b_index];
default: default:
assert(0); assert(0);
return NULL; return NULL;
@@ -790,18 +789,17 @@ static void encode_b(VP9_COMP *cpi, const TileInfo *const tile,
int output_enabled, BLOCK_SIZE bsize, int sub_index) { int output_enabled, BLOCK_SIZE bsize, int sub_index) {
VP9_COMMON *const cm = &cpi->common; VP9_COMMON *const cm = &cpi->common;
MACROBLOCK *const x = &cpi->mb; MACROBLOCK *const x = &cpi->mb;
MACROBLOCKD * const xd = &x->e_mbd;
if (mi_row >= cm->mi_rows || mi_col >= cm->mi_cols) if (mi_row >= cm->mi_rows || mi_col >= cm->mi_cols)
return; return;
if (sub_index != -1) if (sub_index != -1)
*get_sb_index(xd, bsize) = sub_index; *get_sb_index(x, bsize) = sub_index;
if (bsize < BLOCK_8X8) { if (bsize < BLOCK_8X8) {
// When ab_index = 0 all sub-blocks are handled, so for ab_index != 0 // When ab_index = 0 all sub-blocks are handled, so for ab_index != 0
// there is nothing to be done. // there is nothing to be done.
if (xd->ab_index > 0) if (x->ab_index > 0)
return; return;
} }
set_offsets(cpi, tile, mi_row, mi_col, bsize); set_offsets(cpi, tile, mi_row, mi_col, bsize);
@@ -821,7 +819,6 @@ static void encode_sb(VP9_COMP *cpi, const TileInfo *const tile,
int output_enabled, BLOCK_SIZE bsize) { int output_enabled, BLOCK_SIZE bsize) {
VP9_COMMON *const cm = &cpi->common; VP9_COMMON *const cm = &cpi->common;
MACROBLOCK *const x = &cpi->mb; MACROBLOCK *const x = &cpi->mb;
MACROBLOCKD * const xd = &x->e_mbd;
BLOCK_SIZE c1 = BLOCK_8X8; BLOCK_SIZE c1 = BLOCK_8X8;
const int bsl = b_width_log2(bsize), bs = (1 << bsl) / 4; const int bsl = b_width_log2(bsize), bs = (1 << bsl) / 4;
int pl = 0; int pl = 0;
@@ -867,7 +864,7 @@ static void encode_sb(VP9_COMP *cpi, const TileInfo *const tile,
for (i = 0; i < 4; i++) { for (i = 0; i < 4; i++) {
const int x_idx = i & 1, y_idx = i >> 1; const int x_idx = i & 1, y_idx = i >> 1;
*get_sb_index(xd, subsize) = i; *get_sb_index(x, subsize) = i;
encode_sb(cpi, tile, tp, mi_row + y_idx * bs, mi_col + x_idx * bs, encode_sb(cpi, tile, tp, mi_row + y_idx * bs, mi_col + x_idx * bs,
output_enabled, subsize); output_enabled, subsize);
} }
@@ -996,7 +993,6 @@ static void rd_use_partition(VP9_COMP *cpi,
int do_recon) { int do_recon) {
VP9_COMMON *const cm = &cpi->common; VP9_COMMON *const cm = &cpi->common;
MACROBLOCK *const x = &cpi->mb; MACROBLOCK *const x = &cpi->mb;
MACROBLOCKD *xd = &cpi->mb.e_mbd;
const int mis = cm->mode_info_stride; const int mis = cm->mode_info_stride;
int bsl = b_width_log2(bsize); int bsl = b_width_log2(bsize);
const int num_4x4_blocks_wide = num_4x4_blocks_wide_lookup[bsize]; const int num_4x4_blocks_wide = num_4x4_blocks_wide_lookup[bsize];
@@ -1031,7 +1027,7 @@ static void rd_use_partition(VP9_COMP *cpi,
if (bsize < BLOCK_8X8) { if (bsize < BLOCK_8X8) {
// When ab_index = 0 all sub-blocks are handled, so for ab_index != 0 // When ab_index = 0 all sub-blocks are handled, so for ab_index != 0
// there is nothing to be done. // there is nothing to be done.
if (xd->ab_index != 0) { if (x->ab_index != 0) {
*rate = 0; *rate = 0;
*dist = 0; *dist = 0;
return; return;
@@ -1089,7 +1085,7 @@ static void rd_use_partition(VP9_COMP *cpi,
bsize, get_block_context(x, bsize), INT64_MAX); bsize, get_block_context(x, bsize), INT64_MAX);
break; break;
case PARTITION_HORZ: case PARTITION_HORZ:
*get_sb_index(xd, subsize) = 0; *get_sb_index(x, subsize) = 0;
pick_sb_modes(cpi, tile, mi_row, mi_col, &last_part_rate, &last_part_dist, pick_sb_modes(cpi, tile, mi_row, mi_col, &last_part_rate, &last_part_dist,
subsize, get_block_context(x, subsize), INT64_MAX); subsize, get_block_context(x, subsize), INT64_MAX);
if (last_part_rate != INT_MAX && if (last_part_rate != INT_MAX &&
@@ -1098,7 +1094,7 @@ static void rd_use_partition(VP9_COMP *cpi,
int64_t dt = 0; int64_t dt = 0;
update_state(cpi, get_block_context(x, subsize), subsize, 0); update_state(cpi, get_block_context(x, subsize), subsize, 0);
encode_superblock(cpi, tp, 0, mi_row, mi_col, subsize); encode_superblock(cpi, tp, 0, mi_row, mi_col, subsize);
*get_sb_index(xd, subsize) = 1; *get_sb_index(x, subsize) = 1;
pick_sb_modes(cpi, tile, mi_row + (ms >> 1), mi_col, &rt, &dt, subsize, pick_sb_modes(cpi, tile, mi_row + (ms >> 1), mi_col, &rt, &dt, subsize,
get_block_context(x, subsize), INT64_MAX); get_block_context(x, subsize), INT64_MAX);
if (rt == INT_MAX || dt == INT_MAX) { if (rt == INT_MAX || dt == INT_MAX) {
@@ -1112,7 +1108,7 @@ static void rd_use_partition(VP9_COMP *cpi,
} }
break; break;
case PARTITION_VERT: case PARTITION_VERT:
*get_sb_index(xd, subsize) = 0; *get_sb_index(x, subsize) = 0;
pick_sb_modes(cpi, tile, mi_row, mi_col, &last_part_rate, &last_part_dist, pick_sb_modes(cpi, tile, mi_row, mi_col, &last_part_rate, &last_part_dist,
subsize, get_block_context(x, subsize), INT64_MAX); subsize, get_block_context(x, subsize), INT64_MAX);
if (last_part_rate != INT_MAX && if (last_part_rate != INT_MAX &&
@@ -1121,7 +1117,7 @@ static void rd_use_partition(VP9_COMP *cpi,
int64_t dt = 0; int64_t dt = 0;
update_state(cpi, get_block_context(x, subsize), subsize, 0); update_state(cpi, get_block_context(x, subsize), subsize, 0);
encode_superblock(cpi, tp, 0, mi_row, mi_col, subsize); encode_superblock(cpi, tp, 0, mi_row, mi_col, subsize);
*get_sb_index(xd, subsize) = 1; *get_sb_index(x, subsize) = 1;
pick_sb_modes(cpi, tile, mi_row, mi_col + (ms >> 1), &rt, &dt, subsize, pick_sb_modes(cpi, tile, mi_row, mi_col + (ms >> 1), &rt, &dt, subsize,
get_block_context(x, subsize), INT64_MAX); get_block_context(x, subsize), INT64_MAX);
if (rt == INT_MAX || dt == INT_MAX) { if (rt == INT_MAX || dt == INT_MAX) {
@@ -1147,7 +1143,7 @@ static void rd_use_partition(VP9_COMP *cpi,
if ((mi_row + y_idx >= cm->mi_rows) || (mi_col + x_idx >= cm->mi_cols)) if ((mi_row + y_idx >= cm->mi_rows) || (mi_col + x_idx >= cm->mi_cols))
continue; continue;
*get_sb_index(xd, subsize) = i; *get_sb_index(x, subsize) = i;
rd_use_partition(cpi, tile, mi_8x8 + jj * bss * mis + ii * bss, tp, rd_use_partition(cpi, tile, mi_8x8 + jj * bss * mis + ii * bss, tp,
mi_row + y_idx, mi_col + x_idx, subsize, &rt, &dt, mi_row + y_idx, mi_col + x_idx, subsize, &rt, &dt,
@@ -1188,11 +1184,10 @@ static void rd_use_partition(VP9_COMP *cpi,
ENTROPY_CONTEXT l[16 * MAX_MB_PLANE], a[16 * MAX_MB_PLANE]; ENTROPY_CONTEXT l[16 * MAX_MB_PLANE], a[16 * MAX_MB_PLANE];
PARTITION_CONTEXT sl[8], sa[8]; PARTITION_CONTEXT sl[8], sa[8];
if ((mi_row + y_idx >= cm->mi_rows) if ((mi_row + y_idx >= cm->mi_rows) || (mi_col + x_idx >= cm->mi_cols))
|| (mi_col + x_idx >= cm->mi_cols))
continue; continue;
*get_sb_index(xd, split_subsize) = i; *get_sb_index(x, split_subsize) = i;
*get_sb_partitioning(x, bsize) = split_subsize; *get_sb_partitioning(x, bsize) = split_subsize;
*get_sb_partitioning(x, split_subsize) = split_subsize; *get_sb_partitioning(x, split_subsize) = split_subsize;
@@ -1372,7 +1367,6 @@ static void rd_auto_partition_range(VP9_COMP *cpi, const TileInfo *const tile,
static void compute_fast_motion_search_level(VP9_COMP *cpi, BLOCK_SIZE bsize) { static void compute_fast_motion_search_level(VP9_COMP *cpi, BLOCK_SIZE bsize) {
VP9_COMMON *const cm = &cpi->common; VP9_COMMON *const cm = &cpi->common;
MACROBLOCK *const x = &cpi->mb; MACROBLOCK *const x = &cpi->mb;
MACROBLOCKD *const xd = &x->e_mbd;
// Only use 8x8 result for non HD videos. // Only use 8x8 result for non HD videos.
// int use_8x8 = (MIN(cpi->common.width, cpi->common.height) < 720) ? 1 : 0; // int use_8x8 = (MIN(cpi->common.width, cpi->common.height) < 720) ? 1 : 0;
@@ -1385,9 +1379,9 @@ static void compute_fast_motion_search_level(VP9_COMP *cpi, BLOCK_SIZE bsize) {
PICK_MODE_CONTEXT *block_context = NULL; PICK_MODE_CONTEXT *block_context = NULL;
if (bsize == BLOCK_16X16) { if (bsize == BLOCK_16X16) {
block_context = x->sb8x8_context[xd->sb_index][xd->mb_index]; block_context = x->sb8x8_context[x->sb_index][x->mb_index];
} else if (bsize == BLOCK_32X32) { } else if (bsize == BLOCK_32X32) {
block_context = x->mb_context[xd->sb_index]; block_context = x->mb_context[x->sb_index];
} else if (bsize == BLOCK_64X64) { } else if (bsize == BLOCK_64X64) {
block_context = x->sb32_context; block_context = x->sb32_context;
} }
@@ -1477,7 +1471,6 @@ static void rd_pick_partition(VP9_COMP *cpi, const TileInfo *const tile,
int64_t *dist, int do_recon, int64_t best_rd) { int64_t *dist, int do_recon, int64_t best_rd) {
VP9_COMMON *const cm = &cpi->common; VP9_COMMON *const cm = &cpi->common;
MACROBLOCK *const x = &cpi->mb; MACROBLOCK *const x = &cpi->mb;
MACROBLOCKD * const xd = &x->e_mbd;
const int ms = num_8x8_blocks_wide_lookup[bsize] / 2; const int ms = num_8x8_blocks_wide_lookup[bsize] / 2;
ENTROPY_CONTEXT l[16 * MAX_MB_PLANE], a[16 * MAX_MB_PLANE]; ENTROPY_CONTEXT l[16 * MAX_MB_PLANE], a[16 * MAX_MB_PLANE];
PARTITION_CONTEXT sl[8], sa[8]; PARTITION_CONTEXT sl[8], sa[8];
@@ -1503,7 +1496,7 @@ static void rd_pick_partition(VP9_COMP *cpi, const TileInfo *const tile,
if (bsize < BLOCK_8X8) { if (bsize < BLOCK_8X8) {
// When ab_index = 0 all sub-blocks are handled, so for ab_index != 0 // When ab_index = 0 all sub-blocks are handled, so for ab_index != 0
// there is nothing to be done. // there is nothing to be done.
if (xd->ab_index != 0) { if (x->ab_index != 0) {
*rate = 0; *rate = 0;
*dist = 0; *dist = 0;
return; return;
@@ -1601,7 +1594,7 @@ static void rd_pick_partition(VP9_COMP *cpi, const TileInfo *const tile,
if (mi_row + y_idx >= cm->mi_rows || mi_col + x_idx >= cm->mi_cols) if (mi_row + y_idx >= cm->mi_rows || mi_col + x_idx >= cm->mi_cols)
continue; continue;
*get_sb_index(xd, subsize) = i; *get_sb_index(x, subsize) = i;
if (cpi->sf.adaptive_motion_search) if (cpi->sf.adaptive_motion_search)
load_pred_mv(x, get_block_context(x, bsize)); load_pred_mv(x, get_block_context(x, bsize));
rd_pick_partition(cpi, tile, tp, mi_row + y_idx, mi_col + x_idx, subsize, rd_pick_partition(cpi, tile, tp, mi_row + y_idx, mi_col + x_idx, subsize,
@@ -1648,7 +1641,7 @@ static void rd_pick_partition(VP9_COMP *cpi, const TileInfo *const tile,
// PARTITION_HORZ // PARTITION_HORZ
if (partition_horz_allowed && do_rect) { if (partition_horz_allowed && do_rect) {
subsize = get_subsize(bsize, PARTITION_HORZ); subsize = get_subsize(bsize, PARTITION_HORZ);
*get_sb_index(xd, subsize) = 0; *get_sb_index(x, subsize) = 0;
if (cpi->sf.adaptive_motion_search) if (cpi->sf.adaptive_motion_search)
load_pred_mv(x, get_block_context(x, bsize)); load_pred_mv(x, get_block_context(x, bsize));
pick_sb_modes(cpi, tile, mi_row, mi_col, &sum_rate, &sum_dist, subsize, pick_sb_modes(cpi, tile, mi_row, mi_col, &sum_rate, &sum_dist, subsize,
@@ -1659,7 +1652,7 @@ static void rd_pick_partition(VP9_COMP *cpi, const TileInfo *const tile,
update_state(cpi, get_block_context(x, subsize), subsize, 0); update_state(cpi, get_block_context(x, subsize), subsize, 0);
encode_superblock(cpi, tp, 0, mi_row, mi_col, subsize); encode_superblock(cpi, tp, 0, mi_row, mi_col, subsize);
*get_sb_index(xd, subsize) = 1; *get_sb_index(x, subsize) = 1;
if (cpi->sf.adaptive_motion_search) if (cpi->sf.adaptive_motion_search)
load_pred_mv(x, get_block_context(x, bsize)); load_pred_mv(x, get_block_context(x, bsize));
pick_sb_modes(cpi, tile, mi_row + ms, mi_col, &this_rate, pick_sb_modes(cpi, tile, mi_row + ms, mi_col, &this_rate,
@@ -1693,7 +1686,7 @@ static void rd_pick_partition(VP9_COMP *cpi, const TileInfo *const tile,
if (partition_vert_allowed && do_rect) { if (partition_vert_allowed && do_rect) {
subsize = get_subsize(bsize, PARTITION_VERT); subsize = get_subsize(bsize, PARTITION_VERT);
*get_sb_index(xd, subsize) = 0; *get_sb_index(x, subsize) = 0;
if (cpi->sf.adaptive_motion_search) if (cpi->sf.adaptive_motion_search)
load_pred_mv(x, get_block_context(x, bsize)); load_pred_mv(x, get_block_context(x, bsize));
pick_sb_modes(cpi, tile, mi_row, mi_col, &sum_rate, &sum_dist, subsize, pick_sb_modes(cpi, tile, mi_row, mi_col, &sum_rate, &sum_dist, subsize,
@@ -1703,7 +1696,7 @@ static void rd_pick_partition(VP9_COMP *cpi, const TileInfo *const tile,
update_state(cpi, get_block_context(x, subsize), subsize, 0); update_state(cpi, get_block_context(x, subsize), subsize, 0);
encode_superblock(cpi, tp, 0, mi_row, mi_col, subsize); encode_superblock(cpi, tp, 0, mi_row, mi_col, subsize);
*get_sb_index(xd, subsize) = 1; *get_sb_index(x, subsize) = 1;
if (cpi->sf.adaptive_motion_search) if (cpi->sf.adaptive_motion_search)
load_pred_mv(x, get_block_context(x, bsize)); load_pred_mv(x, get_block_context(x, bsize));
pick_sb_modes(cpi, tile, mi_row, mi_col + ms, &this_rate, pick_sb_modes(cpi, tile, mi_row, mi_col + ms, &this_rate,

View File

@@ -1488,34 +1488,33 @@ static void free_mode_context(PICK_MODE_CONTEXT *ctx) {
static void init_pick_mode_context(VP9_COMP *cpi) { static void init_pick_mode_context(VP9_COMP *cpi) {
int i; int i;
MACROBLOCK *x = &cpi->mb; VP9_COMMON *const cm = &cpi->common;
MACROBLOCKD *xd = &x->e_mbd; MACROBLOCK *const x = &cpi->mb;
VP9_COMMON *cm = &cpi->common;
for (i = 0; i < BLOCK_SIZES; ++i) { for (i = 0; i < BLOCK_SIZES; ++i) {
const int num_4x4_w = num_4x4_blocks_wide_lookup[i]; const int num_4x4_w = num_4x4_blocks_wide_lookup[i];
const int num_4x4_h = num_4x4_blocks_high_lookup[i]; const int num_4x4_h = num_4x4_blocks_high_lookup[i];
const int num_4x4_blk = MAX(4, num_4x4_w * num_4x4_h); const int num_4x4_blk = MAX(4, num_4x4_w * num_4x4_h);
if (i < BLOCK_16X16) { if (i < BLOCK_16X16) {
for (xd->sb_index = 0; xd->sb_index < 4; ++xd->sb_index) { for (x->sb_index = 0; x->sb_index < 4; ++x->sb_index) {
for (xd->mb_index = 0; xd->mb_index < 4; ++xd->mb_index) { for (x->mb_index = 0; x->mb_index < 4; ++x->mb_index) {
for (xd->b_index = 0; xd->b_index < 16 / num_4x4_blk; ++xd->b_index) { for (x->b_index = 0; x->b_index < 16 / num_4x4_blk; ++x->b_index) {
PICK_MODE_CONTEXT *ctx = get_block_context(x, i); PICK_MODE_CONTEXT *ctx = get_block_context(x, i);
alloc_mode_context(cm, num_4x4_blk, ctx); alloc_mode_context(cm, num_4x4_blk, ctx);
} }
} }
} }
} else if (i < BLOCK_32X32) { } else if (i < BLOCK_32X32) {
for (xd->sb_index = 0; xd->sb_index < 4; ++xd->sb_index) { for (x->sb_index = 0; x->sb_index < 4; ++x->sb_index) {
for (xd->mb_index = 0; xd->mb_index < 64 / num_4x4_blk; for (x->mb_index = 0; x->mb_index < 64 / num_4x4_blk; ++x->mb_index) {
++xd->mb_index) {
PICK_MODE_CONTEXT *ctx = get_block_context(x, i); PICK_MODE_CONTEXT *ctx = get_block_context(x, i);
ctx->num_4x4_blk = num_4x4_blk; ctx->num_4x4_blk = num_4x4_blk;
alloc_mode_context(cm, num_4x4_blk, ctx); alloc_mode_context(cm, num_4x4_blk, ctx);
} }
} }
} else if (i < BLOCK_64X64) { } else if (i < BLOCK_64X64) {
for (xd->sb_index = 0; xd->sb_index < 256 / num_4x4_blk; ++xd->sb_index) { for (x->sb_index = 0; x->sb_index < 256 / num_4x4_blk; ++x->sb_index) {
PICK_MODE_CONTEXT *ctx = get_block_context(x, i); PICK_MODE_CONTEXT *ctx = get_block_context(x, i);
ctx->num_4x4_blk = num_4x4_blk; ctx->num_4x4_blk = num_4x4_blk;
alloc_mode_context(cm, num_4x4_blk, ctx); alloc_mode_context(cm, num_4x4_blk, ctx);
@@ -1530,31 +1529,29 @@ static void init_pick_mode_context(VP9_COMP *cpi) {
static void free_pick_mode_context(MACROBLOCK *x) { static void free_pick_mode_context(MACROBLOCK *x) {
int i; int i;
MACROBLOCKD *xd = &x->e_mbd;
for (i = 0; i < BLOCK_SIZES; ++i) { for (i = 0; i < BLOCK_SIZES; ++i) {
const int num_4x4_w = num_4x4_blocks_wide_lookup[i]; const int num_4x4_w = num_4x4_blocks_wide_lookup[i];
const int num_4x4_h = num_4x4_blocks_high_lookup[i]; const int num_4x4_h = num_4x4_blocks_high_lookup[i];
const int num_4x4_blk = MAX(4, num_4x4_w * num_4x4_h); const int num_4x4_blk = MAX(4, num_4x4_w * num_4x4_h);
if (i < BLOCK_16X16) { if (i < BLOCK_16X16) {
for (xd->sb_index = 0; xd->sb_index < 4; ++xd->sb_index) { for (x->sb_index = 0; x->sb_index < 4; ++x->sb_index) {
for (xd->mb_index = 0; xd->mb_index < 4; ++xd->mb_index) { for (x->mb_index = 0; x->mb_index < 4; ++x->mb_index) {
for (xd->b_index = 0; xd->b_index < 16 / num_4x4_blk; ++xd->b_index) { for (x->b_index = 0; x->b_index < 16 / num_4x4_blk; ++x->b_index) {
PICK_MODE_CONTEXT *ctx = get_block_context(x, i); PICK_MODE_CONTEXT *ctx = get_block_context(x, i);
free_mode_context(ctx); free_mode_context(ctx);
} }
} }
} }
} else if (i < BLOCK_32X32) { } else if (i < BLOCK_32X32) {
for (xd->sb_index = 0; xd->sb_index < 4; ++xd->sb_index) { for (x->sb_index = 0; x->sb_index < 4; ++x->sb_index) {
for (xd->mb_index = 0; xd->mb_index < 64 / num_4x4_blk; for (x->mb_index = 0; x->mb_index < 64 / num_4x4_blk; ++x->mb_index) {
++xd->mb_index) {
PICK_MODE_CONTEXT *ctx = get_block_context(x, i); PICK_MODE_CONTEXT *ctx = get_block_context(x, i);
free_mode_context(ctx); free_mode_context(ctx);
} }
} }
} else if (i < BLOCK_64X64) { } else if (i < BLOCK_64X64) {
for (xd->sb_index = 0; xd->sb_index < 256 / num_4x4_blk; ++xd->sb_index) { for (x->sb_index = 0; x->sb_index < 256 / num_4x4_blk; ++x->sb_index) {
PICK_MODE_CONTEXT *ctx = get_block_context(x, i); PICK_MODE_CONTEXT *ctx = get_block_context(x, i);
free_mode_context(ctx); free_mode_context(ctx);
} }

View File

@@ -3180,8 +3180,8 @@ int64_t vp9_rd_pick_inter_mode_sb(VP9_COMP *cpi, MACROBLOCK *x,
case BLOCK_32X32: case BLOCK_32X32:
for (i = 0; i < 4; i++) { for (i = 0; i < 4; i++) {
ref_frame_mask |= ref_frame_mask |=
x->mb_context[xd->sb_index][i].frames_with_high_error; x->mb_context[x->sb_index][i].frames_with_high_error;
mode_mask |= x->mb_context[xd->sb_index][i].modes_with_high_error; mode_mask |= x->mb_context[x->sb_index][i].modes_with_high_error;
} }
break; break;
default: default: