Commit 00491e0d authored by Angie Chiang's avatar Angie Chiang
Browse files

Avoid redundant tx_type search

allow_txk_type doesn't contains all the logic of using pre
assigned tx_type or doing the tx_type search.
Here we use get_tx_type to avoid redundant tx_type search.

Change-Id: I09b6bcc60fbe15f0d78689b22d834f95b62bd99a
parent 65a39bb4
......@@ -707,13 +707,6 @@ void av1_write_txb_probs(AV1_COMP *cpi, aom_writer *w) {
for (tx_size = TX_4X4; tx_size <= max_tx_size; ++tx_size)
write_txb_probs(w, cpi, tx_size);
static INLINE int allow_txk_type(MACROBLOCKD *xd, TX_SIZE tx_size, int plane) {
if (plane != 0 || tx_size == TX_32X32 ||
return 0;
return 1;
int64_t av1_search_txk_type(const AV1_COMP *cpi, MACROBLOCK *x, int plane,
int block, int blk_row, int blk_col,
......@@ -729,12 +722,18 @@ int64_t av1_search_txk_type(const AV1_COMP *cpi, MACROBLOCK *x, int plane,
int64_t best_rd = INT64_MAX;
int best_eob = tx_size_2d[tx_size];
const int coeff_ctx = combine_entropy_contexts(*a, *l);
if (!allow_txk_type(xd, tx_size, plane)) txk_end = DCT_DCT;
TX_TYPE tx_type;
for (tx_type = txk_start; tx_type <= txk_end; ++tx_type) {
if (plane == 0) mbmi->txk_type[block] = tx_type;
TX_TYPE ref_tx_type =
get_tx_type(get_plane_type(plane), xd, block, tx_size);
if (tx_type != ref_tx_type) {
// use get_tx_type() to check if the tx_type is valid for the current mode
// if it's not, we skip it here.
RD_STATS this_rd_stats;
if (plane == 0) mbmi->txk_type[block] = tx_type;
av1_xform_quant(cm, x, plane, block, blk_row, blk_col, plane_bsize, tx_size,
coeff_ctx, AV1_XFORM_QUANT_FP);
if (x->plane[plane].eobs[block] && !xd->lossless[mbmi->segment_id])
Supports Markdown
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