Message ID | GV1SPRMB005209878DB1BFA5A29BDF4C8F272@GV1SPRMB0052.EURP250.PROD.OUTLOOK.COM |
---|---|
State | Accepted |
Commit | c1cdaef5cef731412fb5955206e435a171f778de |
Headers | show |
Series | [FFmpeg-devel] avcodec/hevc_cabac: Let compiler count offsets | expand |
Context | Check | Description |
---|---|---|
yinshiyou/make_loongarch64 | success | Make finished |
yinshiyou/make_fate_loongarch64 | success | Make fate finished |
andriy/make_x86 | success | Make finished |
andriy/make_fate_x86 | success | Make fate finished |
Andreas Rheinhardt: > This is easily possible with an X macro. > Using an enum for the offsets also allows to remove > two arrays which are not really needed and will typically > be optimized away by the compiler: The first just exists > to count the number of syntax elements*, the second one > exists to get offset[CONSTANT]. These constants were > of type enum SyntaxElement and this enum was only used > in hevc_cabac.c (although it was declared in hevcdec.h); > it is now no longer needed at all and has therefore been > removed. > > The first of these arrays led to a warning from Clang > which is fixed by this commit: > warning: variable 'num_bins_in_se' is not needed and will > not be emitted [-Wunneeded-internal-declaration] > > *: One could also just added a trailing SYNTAX_ELEMENT_NB > to the SyntaxElement enum for this purpose. > > Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com> > --- Will apply this patch tomorrow unless there are objections. > libavcodec/hevc_cabac.c | 248 ++++++++++++++++------------------------ > libavcodec/hevcdec.h | 52 --------- > 2 files changed, 100 insertions(+), 200 deletions(-) > > diff --git a/libavcodec/hevc_cabac.c b/libavcodec/hevc_cabac.c > index 6b38da84bd..63ffb3d37c 100644 > --- a/libavcodec/hevc_cabac.c > +++ b/libavcodec/hevc_cabac.c > @@ -31,114 +31,66 @@ > > #define CABAC_MAX_BIN 31 > > -/** > - * number of bin by SyntaxElement. > - */ > -static const int8_t num_bins_in_se[] = { > - 1, // sao_merge_flag > - 1, // sao_type_idx > - 0, // sao_eo_class > - 0, // sao_band_position > - 0, // sao_offset_abs > - 0, // sao_offset_sign > - 0, // end_of_slice_flag > - 3, // split_coding_unit_flag > - 1, // cu_transquant_bypass_flag > - 3, // skip_flag > - 3, // cu_qp_delta > - 1, // pred_mode > - 4, // part_mode > - 0, // pcm_flag > - 1, // prev_intra_luma_pred_mode > - 0, // mpm_idx > - 0, // rem_intra_luma_pred_mode > - 2, // intra_chroma_pred_mode > - 1, // merge_flag > - 1, // merge_idx > - 5, // inter_pred_idc > - 2, // ref_idx_l0 > - 2, // ref_idx_l1 > - 2, // abs_mvd_greater0_flag > - 2, // abs_mvd_greater1_flag > - 0, // abs_mvd_minus2 > - 0, // mvd_sign_flag > - 1, // mvp_lx_flag > - 1, // no_residual_data_flag > - 3, // split_transform_flag > - 2, // cbf_luma > - 5, // cbf_cb, cbf_cr > - 2, // transform_skip_flag[][] > - 2, // explicit_rdpcm_flag[][] > - 2, // explicit_rdpcm_dir_flag[][] > - 18, // last_significant_coeff_x_prefix > - 18, // last_significant_coeff_y_prefix > - 0, // last_significant_coeff_x_suffix > - 0, // last_significant_coeff_y_suffix > - 4, // significant_coeff_group_flag > - 44, // significant_coeff_flag > - 24, // coeff_abs_level_greater1_flag > - 6, // coeff_abs_level_greater2_flag > - 0, // coeff_abs_level_remaining > - 0, // coeff_sign_flag > - 8, // log2_res_scale_abs > - 2, // res_scale_sign_flag > - 1, // cu_chroma_qp_offset_flag > - 1, // cu_chroma_qp_offset_idx > -}; > +// ELEM(NAME, NUM_BINS) > +#define CABAC_ELEMS(ELEM) \ > + ELEM(SAO_MERGE_FLAG, 1) \ > + ELEM(SAO_TYPE_IDX, 1) \ > + ELEM(SAO_EO_CLASS, 0) \ > + ELEM(SAO_BAND_POSITION, 0) \ > + ELEM(SAO_OFFSET_ABS, 0) \ > + ELEM(SAO_OFFSET_SIGN, 0) \ > + ELEM(END_OF_SLICE_FLAG, 0) \ > + ELEM(SPLIT_CODING_UNIT_FLAG, 3) \ > + ELEM(CU_TRANSQUANT_BYPASS_FLAG, 1) \ > + ELEM(SKIP_FLAG, 3) \ > + ELEM(CU_QP_DELTA, 3) \ > + ELEM(PRED_MODE_FLAG, 1) \ > + ELEM(PART_MODE, 4) \ > + ELEM(PCM_FLAG, 0) \ > + ELEM(PREV_INTRA_LUMA_PRED_FLAG, 1) \ > + ELEM(MPM_IDX, 0) \ > + ELEM(REM_INTRA_LUMA_PRED_MODE, 0) \ > + ELEM(INTRA_CHROMA_PRED_MODE, 2) \ > + ELEM(MERGE_FLAG, 1) \ > + ELEM(MERGE_IDX, 1) \ > + ELEM(INTER_PRED_IDC, 5) \ > + ELEM(REF_IDX_L0, 2) \ > + ELEM(REF_IDX_L1, 2) \ > + ELEM(ABS_MVD_GREATER0_FLAG, 2) \ > + ELEM(ABS_MVD_GREATER1_FLAG, 2) \ > + ELEM(ABS_MVD_MINUS2, 0) \ > + ELEM(MVD_SIGN_FLAG, 0) \ > + ELEM(MVP_LX_FLAG, 1) \ > + ELEM(NO_RESIDUAL_DATA_FLAG, 1) \ > + ELEM(SPLIT_TRANSFORM_FLAG, 3) \ > + ELEM(CBF_LUMA, 2) \ > + ELEM(CBF_CB_CR, 5) \ > + ELEM(TRANSFORM_SKIP_FLAG, 2) \ > + ELEM(EXPLICIT_RDPCM_FLAG, 2) \ > + ELEM(EXPLICIT_RDPCM_DIR_FLAG, 2) \ > + ELEM(LAST_SIGNIFICANT_COEFF_X_PREFIX, 18) \ > + ELEM(LAST_SIGNIFICANT_COEFF_Y_PREFIX, 18) \ > + ELEM(LAST_SIGNIFICANT_COEFF_X_SUFFIX, 0) \ > + ELEM(LAST_SIGNIFICANT_COEFF_Y_SUFFIX, 0) \ > + ELEM(SIGNIFICANT_COEFF_GROUP_FLAG, 4) \ > + ELEM(SIGNIFICANT_COEFF_FLAG, 44) \ > + ELEM(COEFF_ABS_LEVEL_GREATER1_FLAG, 24) \ > + ELEM(COEFF_ABS_LEVEL_GREATER2_FLAG, 6) \ > + ELEM(COEFF_ABS_LEVEL_REMAINING, 0) \ > + ELEM(COEFF_SIGN_FLAG, 0) \ > + ELEM(LOG2_RES_SCALE_ABS, 8) \ > + ELEM(RES_SCALE_SIGN_FLAG, 2) \ > + ELEM(CU_CHROMA_QP_OFFSET_FLAG, 1) \ > + ELEM(CU_CHROMA_QP_OFFSET_IDX, 1) \ > > /** > - * Offset to ctxIdx 0 in init_values and states, indexed by SyntaxElement. > + * Offset to ctxIdx 0 in init_values and states. > */ > -static const int elem_offset[sizeof(num_bins_in_se)] = { > - 0, // sao_merge_flag > - 1, // sao_type_idx > - 2, // sao_eo_class > - 2, // sao_band_position > - 2, // sao_offset_abs > - 2, // sao_offset_sign > - 2, // end_of_slice_flag > - 2, // split_coding_unit_flag > - 5, // cu_transquant_bypass_flag > - 6, // skip_flag > - 9, // cu_qp_delta > - 12, // pred_mode > - 13, // part_mode > - 17, // pcm_flag > - 17, // prev_intra_luma_pred_mode > - 18, // mpm_idx > - 18, // rem_intra_luma_pred_mode > - 18, // intra_chroma_pred_mode > - 20, // merge_flag > - 21, // merge_idx > - 22, // inter_pred_idc > - 27, // ref_idx_l0 > - 29, // ref_idx_l1 > - 31, // abs_mvd_greater0_flag > - 33, // abs_mvd_greater1_flag > - 35, // abs_mvd_minus2 > - 35, // mvd_sign_flag > - 35, // mvp_lx_flag > - 36, // no_residual_data_flag > - 37, // split_transform_flag > - 40, // cbf_luma > - 42, // cbf_cb, cbf_cr > - 47, // transform_skip_flag[][] > - 49, // explicit_rdpcm_flag[][] > - 51, // explicit_rdpcm_dir_flag[][] > - 53, // last_significant_coeff_x_prefix > - 71, // last_significant_coeff_y_prefix > - 89, // last_significant_coeff_x_suffix > - 89, // last_significant_coeff_y_suffix > - 89, // significant_coeff_group_flag > - 93, // significant_coeff_flag > - 137, // coeff_abs_level_greater1_flag > - 161, // coeff_abs_level_greater2_flag > - 167, // coeff_abs_level_remaining > - 167, // coeff_sign_flag > - 167, // log2_res_scale_abs > - 175, // res_scale_sign_flag > - 177, // cu_chroma_qp_offset_flag > - 178, // cu_chroma_qp_offset_idx > +enum { > +#define OFFSET(NAME, NUM_BINS) \ > + NAME ## _OFFSET, \ > + NAME ## _END = NAME ## _OFFSET + NUM_BINS - 1, > +CABAC_ELEMS(OFFSET) > }; > > #define CNU 154 > @@ -570,12 +522,12 @@ int ff_hevc_cabac_init(HEVCLocalContext *lc, int ctb_addr_ts) > > int ff_hevc_sao_merge_flag_decode(HEVCLocalContext *lc) > { > - return GET_CABAC(elem_offset[SAO_MERGE_FLAG]); > + return GET_CABAC(SAO_MERGE_FLAG_OFFSET); > } > > int ff_hevc_sao_type_idx_decode(HEVCLocalContext *lc) > { > - if (!GET_CABAC(elem_offset[SAO_TYPE_IDX])) > + if (!GET_CABAC(SAO_TYPE_IDX_OFFSET)) > return 0; > > if (!get_cabac_bypass(&lc->cc)) > @@ -622,7 +574,7 @@ int ff_hevc_end_of_slice_flag_decode(HEVCLocalContext *lc) > > int ff_hevc_cu_transquant_bypass_flag_decode(HEVCLocalContext *lc) > { > - return GET_CABAC(elem_offset[CU_TRANSQUANT_BYPASS_FLAG]); > + return GET_CABAC(CU_TRANSQUANT_BYPASS_FLAG_OFFSET); > } > > int ff_hevc_skip_flag_decode(HEVCLocalContext *lc, int x0, int y0, int x_cb, int y_cb) > @@ -638,7 +590,7 @@ int ff_hevc_skip_flag_decode(HEVCLocalContext *lc, int x0, int y0, int x_cb, int > if (lc->ctb_up_flag || y0b) > inc += !!SAMPLE_CTB(s->skip_flag, x_cb, y_cb - 1); > > - return GET_CABAC(elem_offset[SKIP_FLAG] + inc); > + return GET_CABAC(SKIP_FLAG_OFFSET + inc); > } > > int ff_hevc_cu_qp_delta_abs(HEVCLocalContext *lc) > @@ -647,7 +599,7 @@ int ff_hevc_cu_qp_delta_abs(HEVCLocalContext *lc) > int suffix_val = 0; > int inc = 0; > > - while (prefix_val < 5 && GET_CABAC(elem_offset[CU_QP_DELTA] + inc)) { > + while (prefix_val < 5 && GET_CABAC(CU_QP_DELTA_OFFSET + inc)) { > prefix_val++; > inc = 1; > } > @@ -675,7 +627,7 @@ int ff_hevc_cu_qp_delta_sign_flag(HEVCLocalContext *lc) > > int ff_hevc_cu_chroma_qp_offset_flag(HEVCLocalContext *lc) > { > - return GET_CABAC(elem_offset[CU_CHROMA_QP_OFFSET_FLAG]); > + return GET_CABAC(CU_CHROMA_QP_OFFSET_FLAG_OFFSET); > } > > int ff_hevc_cu_chroma_qp_offset_idx(HEVCLocalContext *lc) > @@ -683,7 +635,7 @@ int ff_hevc_cu_chroma_qp_offset_idx(HEVCLocalContext *lc) > int c_max= FFMAX(5, lc->parent->ps.pps->chroma_qp_offset_list_len_minus1); > int i = 0; > > - while (i < c_max && GET_CABAC(elem_offset[CU_CHROMA_QP_OFFSET_IDX])) > + while (i < c_max && GET_CABAC(CU_CHROMA_QP_OFFSET_IDX_OFFSET)) > i++; > > return i; > @@ -691,7 +643,7 @@ int ff_hevc_cu_chroma_qp_offset_idx(HEVCLocalContext *lc) > > int ff_hevc_pred_mode_decode(HEVCLocalContext *lc) > { > - return GET_CABAC(elem_offset[PRED_MODE_FLAG]); > + return GET_CABAC(PRED_MODE_FLAG_OFFSET); > } > > int ff_hevc_split_coding_unit_flag_decode(HEVCLocalContext *lc, int ct_depth, int x0, int y0) > @@ -712,40 +664,40 @@ int ff_hevc_split_coding_unit_flag_decode(HEVCLocalContext *lc, int ct_depth, in > inc += (depth_left > ct_depth); > inc += (depth_top > ct_depth); > > - return GET_CABAC(elem_offset[SPLIT_CODING_UNIT_FLAG] + inc); > + return GET_CABAC(SPLIT_CODING_UNIT_FLAG_OFFSET + inc); > } > > int ff_hevc_part_mode_decode(HEVCLocalContext *lc, int log2_cb_size) > { > - if (GET_CABAC(elem_offset[PART_MODE])) // 1 > + if (GET_CABAC(PART_MODE_OFFSET)) // 1 > return PART_2Nx2N; > if (log2_cb_size == lc->parent->ps.sps->log2_min_cb_size) { > if (lc->cu.pred_mode == MODE_INTRA) // 0 > return PART_NxN; > - if (GET_CABAC(elem_offset[PART_MODE] + 1)) // 01 > + if (GET_CABAC(PART_MODE_OFFSET + 1)) // 01 > return PART_2NxN; > if (log2_cb_size == 3) // 00 > return PART_Nx2N; > - if (GET_CABAC(elem_offset[PART_MODE] + 2)) // 001 > + if (GET_CABAC(PART_MODE_OFFSET + 2)) // 001 > return PART_Nx2N; > return PART_NxN; // 000 > } > > if (!lc->parent->ps.sps->amp_enabled_flag) { > - if (GET_CABAC(elem_offset[PART_MODE] + 1)) // 01 > + if (GET_CABAC(PART_MODE_OFFSET + 1)) // 01 > return PART_2NxN; > return PART_Nx2N; > } > > - if (GET_CABAC(elem_offset[PART_MODE] + 1)) { // 01X, 01XX > - if (GET_CABAC(elem_offset[PART_MODE] + 3)) // 011 > + if (GET_CABAC(PART_MODE_OFFSET + 1)) { // 01X, 01XX > + if (GET_CABAC(PART_MODE_OFFSET + 3)) // 011 > return PART_2NxN; > if (get_cabac_bypass(&lc->cc)) // 0101 > return PART_2NxnD; > return PART_2NxnU; // 0100 > } > > - if (GET_CABAC(elem_offset[PART_MODE] + 3)) // 001 > + if (GET_CABAC(PART_MODE_OFFSET + 3)) // 001 > return PART_Nx2N; > if (get_cabac_bypass(&lc->cc)) // 0001 > return PART_nRx2N; > @@ -759,7 +711,7 @@ int ff_hevc_pcm_flag_decode(HEVCLocalContext *lc) > > int ff_hevc_prev_intra_luma_pred_flag_decode(HEVCLocalContext *lc) > { > - return GET_CABAC(elem_offset[PREV_INTRA_LUMA_PRED_FLAG]); > + return GET_CABAC(PREV_INTRA_LUMA_PRED_FLAG_OFFSET); > } > > int ff_hevc_mpm_idx_decode(HEVCLocalContext *lc) > @@ -783,7 +735,7 @@ int ff_hevc_rem_intra_luma_pred_mode_decode(HEVCLocalContext *lc) > int ff_hevc_intra_chroma_pred_mode_decode(HEVCLocalContext *lc) > { > int ret; > - if (!GET_CABAC(elem_offset[INTRA_CHROMA_PRED_MODE])) > + if (!GET_CABAC(INTRA_CHROMA_PRED_MODE_OFFSET)) > return 4; > > ret = get_cabac_bypass(&lc->cc) << 1; > @@ -793,7 +745,7 @@ int ff_hevc_intra_chroma_pred_mode_decode(HEVCLocalContext *lc) > > int ff_hevc_merge_idx_decode(HEVCLocalContext *lc) > { > - int i = GET_CABAC(elem_offset[MERGE_IDX]); > + int i = GET_CABAC(MERGE_IDX_OFFSET); > > if (i != 0) { > while (i < lc->parent->sh.max_num_merge_cand-1 && get_cabac_bypass(&lc->cc)) > @@ -804,17 +756,17 @@ int ff_hevc_merge_idx_decode(HEVCLocalContext *lc) > > int ff_hevc_merge_flag_decode(HEVCLocalContext *lc) > { > - return GET_CABAC(elem_offset[MERGE_FLAG]); > + return GET_CABAC(MERGE_FLAG_OFFSET); > } > > int ff_hevc_inter_pred_idc_decode(HEVCLocalContext *lc, int nPbW, int nPbH) > { > if (nPbW + nPbH == 12) > - return GET_CABAC(elem_offset[INTER_PRED_IDC] + 4); > - if (GET_CABAC(elem_offset[INTER_PRED_IDC] + lc->ct_depth)) > + return GET_CABAC(INTER_PRED_IDC_OFFSET + 4); > + if (GET_CABAC(INTER_PRED_IDC_OFFSET + lc->ct_depth)) > return PRED_BI; > > - return GET_CABAC(elem_offset[INTER_PRED_IDC] + 4); > + return GET_CABAC(INTER_PRED_IDC_OFFSET + 4); > } > > int ff_hevc_ref_idx_lx_decode(HEVCLocalContext *lc, int num_ref_idx_lx) > @@ -823,7 +775,7 @@ int ff_hevc_ref_idx_lx_decode(HEVCLocalContext *lc, int num_ref_idx_lx) > int max = num_ref_idx_lx - 1; > int max_ctx = FFMIN(max, 2); > > - while (i < max_ctx && GET_CABAC(elem_offset[REF_IDX_L0] + i)) > + while (i < max_ctx && GET_CABAC(REF_IDX_L0_OFFSET + i)) > i++; > if (i == 2) { > while (i < max && get_cabac_bypass(&lc->cc)) > @@ -835,22 +787,22 @@ int ff_hevc_ref_idx_lx_decode(HEVCLocalContext *lc, int num_ref_idx_lx) > > int ff_hevc_mvp_lx_flag_decode(HEVCLocalContext *lc) > { > - return GET_CABAC(elem_offset[MVP_LX_FLAG]); > + return GET_CABAC(MVP_LX_FLAG_OFFSET); > } > > int ff_hevc_no_residual_syntax_flag_decode(HEVCLocalContext *lc) > { > - return GET_CABAC(elem_offset[NO_RESIDUAL_DATA_FLAG]); > + return GET_CABAC(NO_RESIDUAL_DATA_FLAG_OFFSET); > } > > static av_always_inline int abs_mvd_greater0_flag_decode(HEVCLocalContext *lc) > { > - return GET_CABAC(elem_offset[ABS_MVD_GREATER0_FLAG]); > + return GET_CABAC(ABS_MVD_GREATER0_FLAG_OFFSET); > } > > static av_always_inline int abs_mvd_greater1_flag_decode(HEVCLocalContext *lc) > { > - return GET_CABAC(elem_offset[ABS_MVD_GREATER1_FLAG] + 1); > + return GET_CABAC(ABS_MVD_GREATER1_FLAG_OFFSET + 1); > } > > static av_always_inline int mvd_decode(HEVCLocalContext *lc) > @@ -878,39 +830,39 @@ static av_always_inline int mvd_sign_flag_decode(HEVCLocalContext *lc) > > int ff_hevc_split_transform_flag_decode(HEVCLocalContext *lc, int log2_trafo_size) > { > - return GET_CABAC(elem_offset[SPLIT_TRANSFORM_FLAG] + 5 - log2_trafo_size); > + return GET_CABAC(SPLIT_TRANSFORM_FLAG_OFFSET + 5 - log2_trafo_size); > } > > int ff_hevc_cbf_cb_cr_decode(HEVCLocalContext *lc, int trafo_depth) > { > - return GET_CABAC(elem_offset[CBF_CB_CR] + trafo_depth); > + return GET_CABAC(CBF_CB_CR_OFFSET + trafo_depth); > } > > int ff_hevc_cbf_luma_decode(HEVCLocalContext *lc, int trafo_depth) > { > - return GET_CABAC(elem_offset[CBF_LUMA] + !trafo_depth); > + return GET_CABAC(CBF_LUMA_OFFSET + !trafo_depth); > } > > static int hevc_transform_skip_flag_decode(HEVCLocalContext *lc, int c_idx) > { > - return GET_CABAC(elem_offset[TRANSFORM_SKIP_FLAG] + !!c_idx); > + return GET_CABAC(TRANSFORM_SKIP_FLAG_OFFSET + !!c_idx); > } > > static int explicit_rdpcm_flag_decode(HEVCLocalContext *lc, int c_idx) > { > - return GET_CABAC(elem_offset[EXPLICIT_RDPCM_FLAG] + !!c_idx); > + return GET_CABAC(EXPLICIT_RDPCM_FLAG_OFFSET + !!c_idx); > } > > static int explicit_rdpcm_dir_flag_decode(HEVCLocalContext *lc, int c_idx) > { > - return GET_CABAC(elem_offset[EXPLICIT_RDPCM_DIR_FLAG] + !!c_idx); > + return GET_CABAC(EXPLICIT_RDPCM_DIR_FLAG_OFFSET + !!c_idx); > } > > int ff_hevc_log2_res_scale_abs(HEVCLocalContext *lc, int idx) > { > int i =0; > > - while (i < 4 && GET_CABAC(elem_offset[LOG2_RES_SCALE_ABS] + 4 * idx + i)) > + while (i < 4 && GET_CABAC(LOG2_RES_SCALE_ABS_OFFSET + 4 * idx + i)) > i++; > > return i; > @@ -918,7 +870,7 @@ int ff_hevc_log2_res_scale_abs(HEVCLocalContext *lc, int idx) > > int ff_hevc_res_scale_sign_flag(HEVCLocalContext *lc, int idx) > { > - return GET_CABAC(elem_offset[RES_SCALE_SIGN_FLAG] + idx); > + return GET_CABAC(RES_SCALE_SIGN_FLAG_OFFSET + idx); > } > > static av_always_inline void last_significant_coeff_xy_prefix_decode(HEVCLocalContext *lc, int c_idx, > @@ -936,13 +888,13 @@ static av_always_inline void last_significant_coeff_xy_prefix_decode(HEVCLocalCo > ctx_shift = log2_size - 2; > } > while (i < max && > - GET_CABAC(elem_offset[LAST_SIGNIFICANT_COEFF_X_PREFIX] + (i >> ctx_shift) + ctx_offset)) > + GET_CABAC(LAST_SIGNIFICANT_COEFF_X_PREFIX_OFFSET + (i >> ctx_shift) + ctx_offset)) > i++; > *last_scx_prefix = i; > > i = 0; > while (i < max && > - GET_CABAC(elem_offset[LAST_SIGNIFICANT_COEFF_Y_PREFIX] + (i >> ctx_shift) + ctx_offset)) > + GET_CABAC(LAST_SIGNIFICANT_COEFF_Y_PREFIX_OFFSET + (i >> ctx_shift) + ctx_offset)) > i++; > *last_scy_prefix = i; > } > @@ -965,18 +917,18 @@ static av_always_inline int significant_coeff_group_flag_decode(HEVCLocalContext > > inc = FFMIN(ctx_cg, 1) + (c_idx>0 ? 2 : 0); > > - return GET_CABAC(elem_offset[SIGNIFICANT_COEFF_GROUP_FLAG] + inc); > + return GET_CABAC(SIGNIFICANT_COEFF_GROUP_FLAG_OFFSET + inc); > } > static av_always_inline int significant_coeff_flag_decode(HEVCLocalContext *lc, int x_c, int y_c, > int offset, const uint8_t *ctx_idx_map) > { > int inc = ctx_idx_map[(y_c << 2) + x_c] + offset; > - return GET_CABAC(elem_offset[SIGNIFICANT_COEFF_FLAG] + inc); > + return GET_CABAC(SIGNIFICANT_COEFF_FLAG_OFFSET + inc); > } > > static av_always_inline int significant_coeff_flag_decode_0(HEVCLocalContext *lc, int c_idx, int offset) > { > - return GET_CABAC(elem_offset[SIGNIFICANT_COEFF_FLAG] + offset); > + return GET_CABAC(SIGNIFICANT_COEFF_FLAG_OFFSET + offset); > } > > static av_always_inline int coeff_abs_level_greater1_flag_decode(HEVCLocalContext *lc, int c_idx, int inc) > @@ -985,7 +937,7 @@ static av_always_inline int coeff_abs_level_greater1_flag_decode(HEVCLocalContex > if (c_idx > 0) > inc += 16; > > - return GET_CABAC(elem_offset[COEFF_ABS_LEVEL_GREATER1_FLAG] + inc); > + return GET_CABAC(COEFF_ABS_LEVEL_GREATER1_FLAG_OFFSET + inc); > } > > static av_always_inline int coeff_abs_level_greater2_flag_decode(HEVCLocalContext *lc, int c_idx, int inc) > @@ -993,7 +945,7 @@ static av_always_inline int coeff_abs_level_greater2_flag_decode(HEVCLocalContex > if (c_idx > 0) > inc += 4; > > - return GET_CABAC(elem_offset[COEFF_ABS_LEVEL_GREATER2_FLAG] + inc); > + return GET_CABAC(COEFF_ABS_LEVEL_GREATER2_FLAG_OFFSET + inc); > } > > static av_always_inline int coeff_abs_level_remaining_decode(HEVCLocalContext *lc, int rc_rice_param) > diff --git a/libavcodec/hevcdec.h b/libavcodec/hevcdec.h > index c697119891..e65a6180ca 100644 > --- a/libavcodec/hevcdec.h > +++ b/libavcodec/hevcdec.h > @@ -86,58 +86,6 @@ enum RPSType { > NB_RPS_TYPE, > }; > > -enum SyntaxElement { > - SAO_MERGE_FLAG = 0, > - SAO_TYPE_IDX, > - SAO_EO_CLASS, > - SAO_BAND_POSITION, > - SAO_OFFSET_ABS, > - SAO_OFFSET_SIGN, > - END_OF_SLICE_FLAG, > - SPLIT_CODING_UNIT_FLAG, > - CU_TRANSQUANT_BYPASS_FLAG, > - SKIP_FLAG, > - CU_QP_DELTA, > - PRED_MODE_FLAG, > - PART_MODE, > - PCM_FLAG, > - PREV_INTRA_LUMA_PRED_FLAG, > - MPM_IDX, > - REM_INTRA_LUMA_PRED_MODE, > - INTRA_CHROMA_PRED_MODE, > - MERGE_FLAG, > - MERGE_IDX, > - INTER_PRED_IDC, > - REF_IDX_L0, > - REF_IDX_L1, > - ABS_MVD_GREATER0_FLAG, > - ABS_MVD_GREATER1_FLAG, > - ABS_MVD_MINUS2, > - MVD_SIGN_FLAG, > - MVP_LX_FLAG, > - NO_RESIDUAL_DATA_FLAG, > - SPLIT_TRANSFORM_FLAG, > - CBF_LUMA, > - CBF_CB_CR, > - TRANSFORM_SKIP_FLAG, > - EXPLICIT_RDPCM_FLAG, > - EXPLICIT_RDPCM_DIR_FLAG, > - LAST_SIGNIFICANT_COEFF_X_PREFIX, > - LAST_SIGNIFICANT_COEFF_Y_PREFIX, > - LAST_SIGNIFICANT_COEFF_X_SUFFIX, > - LAST_SIGNIFICANT_COEFF_Y_SUFFIX, > - SIGNIFICANT_COEFF_GROUP_FLAG, > - SIGNIFICANT_COEFF_FLAG, > - COEFF_ABS_LEVEL_GREATER1_FLAG, > - COEFF_ABS_LEVEL_GREATER2_FLAG, > - COEFF_ABS_LEVEL_REMAINING, > - COEFF_SIGN_FLAG, > - LOG2_RES_SCALE_ABS, > - RES_SCALE_SIGN_FLAG, > - CU_CHROMA_QP_OFFSET_FLAG, > - CU_CHROMA_QP_OFFSET_IDX, > -}; > - > enum PartMode { > PART_2Nx2N = 0, > PART_2NxN = 1,
diff --git a/libavcodec/hevc_cabac.c b/libavcodec/hevc_cabac.c index 6b38da84bd..63ffb3d37c 100644 --- a/libavcodec/hevc_cabac.c +++ b/libavcodec/hevc_cabac.c @@ -31,114 +31,66 @@ #define CABAC_MAX_BIN 31 -/** - * number of bin by SyntaxElement. - */ -static const int8_t num_bins_in_se[] = { - 1, // sao_merge_flag - 1, // sao_type_idx - 0, // sao_eo_class - 0, // sao_band_position - 0, // sao_offset_abs - 0, // sao_offset_sign - 0, // end_of_slice_flag - 3, // split_coding_unit_flag - 1, // cu_transquant_bypass_flag - 3, // skip_flag - 3, // cu_qp_delta - 1, // pred_mode - 4, // part_mode - 0, // pcm_flag - 1, // prev_intra_luma_pred_mode - 0, // mpm_idx - 0, // rem_intra_luma_pred_mode - 2, // intra_chroma_pred_mode - 1, // merge_flag - 1, // merge_idx - 5, // inter_pred_idc - 2, // ref_idx_l0 - 2, // ref_idx_l1 - 2, // abs_mvd_greater0_flag - 2, // abs_mvd_greater1_flag - 0, // abs_mvd_minus2 - 0, // mvd_sign_flag - 1, // mvp_lx_flag - 1, // no_residual_data_flag - 3, // split_transform_flag - 2, // cbf_luma - 5, // cbf_cb, cbf_cr - 2, // transform_skip_flag[][] - 2, // explicit_rdpcm_flag[][] - 2, // explicit_rdpcm_dir_flag[][] - 18, // last_significant_coeff_x_prefix - 18, // last_significant_coeff_y_prefix - 0, // last_significant_coeff_x_suffix - 0, // last_significant_coeff_y_suffix - 4, // significant_coeff_group_flag - 44, // significant_coeff_flag - 24, // coeff_abs_level_greater1_flag - 6, // coeff_abs_level_greater2_flag - 0, // coeff_abs_level_remaining - 0, // coeff_sign_flag - 8, // log2_res_scale_abs - 2, // res_scale_sign_flag - 1, // cu_chroma_qp_offset_flag - 1, // cu_chroma_qp_offset_idx -}; +// ELEM(NAME, NUM_BINS) +#define CABAC_ELEMS(ELEM) \ + ELEM(SAO_MERGE_FLAG, 1) \ + ELEM(SAO_TYPE_IDX, 1) \ + ELEM(SAO_EO_CLASS, 0) \ + ELEM(SAO_BAND_POSITION, 0) \ + ELEM(SAO_OFFSET_ABS, 0) \ + ELEM(SAO_OFFSET_SIGN, 0) \ + ELEM(END_OF_SLICE_FLAG, 0) \ + ELEM(SPLIT_CODING_UNIT_FLAG, 3) \ + ELEM(CU_TRANSQUANT_BYPASS_FLAG, 1) \ + ELEM(SKIP_FLAG, 3) \ + ELEM(CU_QP_DELTA, 3) \ + ELEM(PRED_MODE_FLAG, 1) \ + ELEM(PART_MODE, 4) \ + ELEM(PCM_FLAG, 0) \ + ELEM(PREV_INTRA_LUMA_PRED_FLAG, 1) \ + ELEM(MPM_IDX, 0) \ + ELEM(REM_INTRA_LUMA_PRED_MODE, 0) \ + ELEM(INTRA_CHROMA_PRED_MODE, 2) \ + ELEM(MERGE_FLAG, 1) \ + ELEM(MERGE_IDX, 1) \ + ELEM(INTER_PRED_IDC, 5) \ + ELEM(REF_IDX_L0, 2) \ + ELEM(REF_IDX_L1, 2) \ + ELEM(ABS_MVD_GREATER0_FLAG, 2) \ + ELEM(ABS_MVD_GREATER1_FLAG, 2) \ + ELEM(ABS_MVD_MINUS2, 0) \ + ELEM(MVD_SIGN_FLAG, 0) \ + ELEM(MVP_LX_FLAG, 1) \ + ELEM(NO_RESIDUAL_DATA_FLAG, 1) \ + ELEM(SPLIT_TRANSFORM_FLAG, 3) \ + ELEM(CBF_LUMA, 2) \ + ELEM(CBF_CB_CR, 5) \ + ELEM(TRANSFORM_SKIP_FLAG, 2) \ + ELEM(EXPLICIT_RDPCM_FLAG, 2) \ + ELEM(EXPLICIT_RDPCM_DIR_FLAG, 2) \ + ELEM(LAST_SIGNIFICANT_COEFF_X_PREFIX, 18) \ + ELEM(LAST_SIGNIFICANT_COEFF_Y_PREFIX, 18) \ + ELEM(LAST_SIGNIFICANT_COEFF_X_SUFFIX, 0) \ + ELEM(LAST_SIGNIFICANT_COEFF_Y_SUFFIX, 0) \ + ELEM(SIGNIFICANT_COEFF_GROUP_FLAG, 4) \ + ELEM(SIGNIFICANT_COEFF_FLAG, 44) \ + ELEM(COEFF_ABS_LEVEL_GREATER1_FLAG, 24) \ + ELEM(COEFF_ABS_LEVEL_GREATER2_FLAG, 6) \ + ELEM(COEFF_ABS_LEVEL_REMAINING, 0) \ + ELEM(COEFF_SIGN_FLAG, 0) \ + ELEM(LOG2_RES_SCALE_ABS, 8) \ + ELEM(RES_SCALE_SIGN_FLAG, 2) \ + ELEM(CU_CHROMA_QP_OFFSET_FLAG, 1) \ + ELEM(CU_CHROMA_QP_OFFSET_IDX, 1) \ /** - * Offset to ctxIdx 0 in init_values and states, indexed by SyntaxElement. + * Offset to ctxIdx 0 in init_values and states. */ -static const int elem_offset[sizeof(num_bins_in_se)] = { - 0, // sao_merge_flag - 1, // sao_type_idx - 2, // sao_eo_class - 2, // sao_band_position - 2, // sao_offset_abs - 2, // sao_offset_sign - 2, // end_of_slice_flag - 2, // split_coding_unit_flag - 5, // cu_transquant_bypass_flag - 6, // skip_flag - 9, // cu_qp_delta - 12, // pred_mode - 13, // part_mode - 17, // pcm_flag - 17, // prev_intra_luma_pred_mode - 18, // mpm_idx - 18, // rem_intra_luma_pred_mode - 18, // intra_chroma_pred_mode - 20, // merge_flag - 21, // merge_idx - 22, // inter_pred_idc - 27, // ref_idx_l0 - 29, // ref_idx_l1 - 31, // abs_mvd_greater0_flag - 33, // abs_mvd_greater1_flag - 35, // abs_mvd_minus2 - 35, // mvd_sign_flag - 35, // mvp_lx_flag - 36, // no_residual_data_flag - 37, // split_transform_flag - 40, // cbf_luma - 42, // cbf_cb, cbf_cr - 47, // transform_skip_flag[][] - 49, // explicit_rdpcm_flag[][] - 51, // explicit_rdpcm_dir_flag[][] - 53, // last_significant_coeff_x_prefix - 71, // last_significant_coeff_y_prefix - 89, // last_significant_coeff_x_suffix - 89, // last_significant_coeff_y_suffix - 89, // significant_coeff_group_flag - 93, // significant_coeff_flag - 137, // coeff_abs_level_greater1_flag - 161, // coeff_abs_level_greater2_flag - 167, // coeff_abs_level_remaining - 167, // coeff_sign_flag - 167, // log2_res_scale_abs - 175, // res_scale_sign_flag - 177, // cu_chroma_qp_offset_flag - 178, // cu_chroma_qp_offset_idx +enum { +#define OFFSET(NAME, NUM_BINS) \ + NAME ## _OFFSET, \ + NAME ## _END = NAME ## _OFFSET + NUM_BINS - 1, +CABAC_ELEMS(OFFSET) }; #define CNU 154 @@ -570,12 +522,12 @@ int ff_hevc_cabac_init(HEVCLocalContext *lc, int ctb_addr_ts) int ff_hevc_sao_merge_flag_decode(HEVCLocalContext *lc) { - return GET_CABAC(elem_offset[SAO_MERGE_FLAG]); + return GET_CABAC(SAO_MERGE_FLAG_OFFSET); } int ff_hevc_sao_type_idx_decode(HEVCLocalContext *lc) { - if (!GET_CABAC(elem_offset[SAO_TYPE_IDX])) + if (!GET_CABAC(SAO_TYPE_IDX_OFFSET)) return 0; if (!get_cabac_bypass(&lc->cc)) @@ -622,7 +574,7 @@ int ff_hevc_end_of_slice_flag_decode(HEVCLocalContext *lc) int ff_hevc_cu_transquant_bypass_flag_decode(HEVCLocalContext *lc) { - return GET_CABAC(elem_offset[CU_TRANSQUANT_BYPASS_FLAG]); + return GET_CABAC(CU_TRANSQUANT_BYPASS_FLAG_OFFSET); } int ff_hevc_skip_flag_decode(HEVCLocalContext *lc, int x0, int y0, int x_cb, int y_cb) @@ -638,7 +590,7 @@ int ff_hevc_skip_flag_decode(HEVCLocalContext *lc, int x0, int y0, int x_cb, int if (lc->ctb_up_flag || y0b) inc += !!SAMPLE_CTB(s->skip_flag, x_cb, y_cb - 1); - return GET_CABAC(elem_offset[SKIP_FLAG] + inc); + return GET_CABAC(SKIP_FLAG_OFFSET + inc); } int ff_hevc_cu_qp_delta_abs(HEVCLocalContext *lc) @@ -647,7 +599,7 @@ int ff_hevc_cu_qp_delta_abs(HEVCLocalContext *lc) int suffix_val = 0; int inc = 0; - while (prefix_val < 5 && GET_CABAC(elem_offset[CU_QP_DELTA] + inc)) { + while (prefix_val < 5 && GET_CABAC(CU_QP_DELTA_OFFSET + inc)) { prefix_val++; inc = 1; } @@ -675,7 +627,7 @@ int ff_hevc_cu_qp_delta_sign_flag(HEVCLocalContext *lc) int ff_hevc_cu_chroma_qp_offset_flag(HEVCLocalContext *lc) { - return GET_CABAC(elem_offset[CU_CHROMA_QP_OFFSET_FLAG]); + return GET_CABAC(CU_CHROMA_QP_OFFSET_FLAG_OFFSET); } int ff_hevc_cu_chroma_qp_offset_idx(HEVCLocalContext *lc) @@ -683,7 +635,7 @@ int ff_hevc_cu_chroma_qp_offset_idx(HEVCLocalContext *lc) int c_max= FFMAX(5, lc->parent->ps.pps->chroma_qp_offset_list_len_minus1); int i = 0; - while (i < c_max && GET_CABAC(elem_offset[CU_CHROMA_QP_OFFSET_IDX])) + while (i < c_max && GET_CABAC(CU_CHROMA_QP_OFFSET_IDX_OFFSET)) i++; return i; @@ -691,7 +643,7 @@ int ff_hevc_cu_chroma_qp_offset_idx(HEVCLocalContext *lc) int ff_hevc_pred_mode_decode(HEVCLocalContext *lc) { - return GET_CABAC(elem_offset[PRED_MODE_FLAG]); + return GET_CABAC(PRED_MODE_FLAG_OFFSET); } int ff_hevc_split_coding_unit_flag_decode(HEVCLocalContext *lc, int ct_depth, int x0, int y0) @@ -712,40 +664,40 @@ int ff_hevc_split_coding_unit_flag_decode(HEVCLocalContext *lc, int ct_depth, in inc += (depth_left > ct_depth); inc += (depth_top > ct_depth); - return GET_CABAC(elem_offset[SPLIT_CODING_UNIT_FLAG] + inc); + return GET_CABAC(SPLIT_CODING_UNIT_FLAG_OFFSET + inc); } int ff_hevc_part_mode_decode(HEVCLocalContext *lc, int log2_cb_size) { - if (GET_CABAC(elem_offset[PART_MODE])) // 1 + if (GET_CABAC(PART_MODE_OFFSET)) // 1 return PART_2Nx2N; if (log2_cb_size == lc->parent->ps.sps->log2_min_cb_size) { if (lc->cu.pred_mode == MODE_INTRA) // 0 return PART_NxN; - if (GET_CABAC(elem_offset[PART_MODE] + 1)) // 01 + if (GET_CABAC(PART_MODE_OFFSET + 1)) // 01 return PART_2NxN; if (log2_cb_size == 3) // 00 return PART_Nx2N; - if (GET_CABAC(elem_offset[PART_MODE] + 2)) // 001 + if (GET_CABAC(PART_MODE_OFFSET + 2)) // 001 return PART_Nx2N; return PART_NxN; // 000 } if (!lc->parent->ps.sps->amp_enabled_flag) { - if (GET_CABAC(elem_offset[PART_MODE] + 1)) // 01 + if (GET_CABAC(PART_MODE_OFFSET + 1)) // 01 return PART_2NxN; return PART_Nx2N; } - if (GET_CABAC(elem_offset[PART_MODE] + 1)) { // 01X, 01XX - if (GET_CABAC(elem_offset[PART_MODE] + 3)) // 011 + if (GET_CABAC(PART_MODE_OFFSET + 1)) { // 01X, 01XX + if (GET_CABAC(PART_MODE_OFFSET + 3)) // 011 return PART_2NxN; if (get_cabac_bypass(&lc->cc)) // 0101 return PART_2NxnD; return PART_2NxnU; // 0100 } - if (GET_CABAC(elem_offset[PART_MODE] + 3)) // 001 + if (GET_CABAC(PART_MODE_OFFSET + 3)) // 001 return PART_Nx2N; if (get_cabac_bypass(&lc->cc)) // 0001 return PART_nRx2N; @@ -759,7 +711,7 @@ int ff_hevc_pcm_flag_decode(HEVCLocalContext *lc) int ff_hevc_prev_intra_luma_pred_flag_decode(HEVCLocalContext *lc) { - return GET_CABAC(elem_offset[PREV_INTRA_LUMA_PRED_FLAG]); + return GET_CABAC(PREV_INTRA_LUMA_PRED_FLAG_OFFSET); } int ff_hevc_mpm_idx_decode(HEVCLocalContext *lc) @@ -783,7 +735,7 @@ int ff_hevc_rem_intra_luma_pred_mode_decode(HEVCLocalContext *lc) int ff_hevc_intra_chroma_pred_mode_decode(HEVCLocalContext *lc) { int ret; - if (!GET_CABAC(elem_offset[INTRA_CHROMA_PRED_MODE])) + if (!GET_CABAC(INTRA_CHROMA_PRED_MODE_OFFSET)) return 4; ret = get_cabac_bypass(&lc->cc) << 1; @@ -793,7 +745,7 @@ int ff_hevc_intra_chroma_pred_mode_decode(HEVCLocalContext *lc) int ff_hevc_merge_idx_decode(HEVCLocalContext *lc) { - int i = GET_CABAC(elem_offset[MERGE_IDX]); + int i = GET_CABAC(MERGE_IDX_OFFSET); if (i != 0) { while (i < lc->parent->sh.max_num_merge_cand-1 && get_cabac_bypass(&lc->cc)) @@ -804,17 +756,17 @@ int ff_hevc_merge_idx_decode(HEVCLocalContext *lc) int ff_hevc_merge_flag_decode(HEVCLocalContext *lc) { - return GET_CABAC(elem_offset[MERGE_FLAG]); + return GET_CABAC(MERGE_FLAG_OFFSET); } int ff_hevc_inter_pred_idc_decode(HEVCLocalContext *lc, int nPbW, int nPbH) { if (nPbW + nPbH == 12) - return GET_CABAC(elem_offset[INTER_PRED_IDC] + 4); - if (GET_CABAC(elem_offset[INTER_PRED_IDC] + lc->ct_depth)) + return GET_CABAC(INTER_PRED_IDC_OFFSET + 4); + if (GET_CABAC(INTER_PRED_IDC_OFFSET + lc->ct_depth)) return PRED_BI; - return GET_CABAC(elem_offset[INTER_PRED_IDC] + 4); + return GET_CABAC(INTER_PRED_IDC_OFFSET + 4); } int ff_hevc_ref_idx_lx_decode(HEVCLocalContext *lc, int num_ref_idx_lx) @@ -823,7 +775,7 @@ int ff_hevc_ref_idx_lx_decode(HEVCLocalContext *lc, int num_ref_idx_lx) int max = num_ref_idx_lx - 1; int max_ctx = FFMIN(max, 2); - while (i < max_ctx && GET_CABAC(elem_offset[REF_IDX_L0] + i)) + while (i < max_ctx && GET_CABAC(REF_IDX_L0_OFFSET + i)) i++; if (i == 2) { while (i < max && get_cabac_bypass(&lc->cc)) @@ -835,22 +787,22 @@ int ff_hevc_ref_idx_lx_decode(HEVCLocalContext *lc, int num_ref_idx_lx) int ff_hevc_mvp_lx_flag_decode(HEVCLocalContext *lc) { - return GET_CABAC(elem_offset[MVP_LX_FLAG]); + return GET_CABAC(MVP_LX_FLAG_OFFSET); } int ff_hevc_no_residual_syntax_flag_decode(HEVCLocalContext *lc) { - return GET_CABAC(elem_offset[NO_RESIDUAL_DATA_FLAG]); + return GET_CABAC(NO_RESIDUAL_DATA_FLAG_OFFSET); } static av_always_inline int abs_mvd_greater0_flag_decode(HEVCLocalContext *lc) { - return GET_CABAC(elem_offset[ABS_MVD_GREATER0_FLAG]); + return GET_CABAC(ABS_MVD_GREATER0_FLAG_OFFSET); } static av_always_inline int abs_mvd_greater1_flag_decode(HEVCLocalContext *lc) { - return GET_CABAC(elem_offset[ABS_MVD_GREATER1_FLAG] + 1); + return GET_CABAC(ABS_MVD_GREATER1_FLAG_OFFSET + 1); } static av_always_inline int mvd_decode(HEVCLocalContext *lc) @@ -878,39 +830,39 @@ static av_always_inline int mvd_sign_flag_decode(HEVCLocalContext *lc) int ff_hevc_split_transform_flag_decode(HEVCLocalContext *lc, int log2_trafo_size) { - return GET_CABAC(elem_offset[SPLIT_TRANSFORM_FLAG] + 5 - log2_trafo_size); + return GET_CABAC(SPLIT_TRANSFORM_FLAG_OFFSET + 5 - log2_trafo_size); } int ff_hevc_cbf_cb_cr_decode(HEVCLocalContext *lc, int trafo_depth) { - return GET_CABAC(elem_offset[CBF_CB_CR] + trafo_depth); + return GET_CABAC(CBF_CB_CR_OFFSET + trafo_depth); } int ff_hevc_cbf_luma_decode(HEVCLocalContext *lc, int trafo_depth) { - return GET_CABAC(elem_offset[CBF_LUMA] + !trafo_depth); + return GET_CABAC(CBF_LUMA_OFFSET + !trafo_depth); } static int hevc_transform_skip_flag_decode(HEVCLocalContext *lc, int c_idx) { - return GET_CABAC(elem_offset[TRANSFORM_SKIP_FLAG] + !!c_idx); + return GET_CABAC(TRANSFORM_SKIP_FLAG_OFFSET + !!c_idx); } static int explicit_rdpcm_flag_decode(HEVCLocalContext *lc, int c_idx) { - return GET_CABAC(elem_offset[EXPLICIT_RDPCM_FLAG] + !!c_idx); + return GET_CABAC(EXPLICIT_RDPCM_FLAG_OFFSET + !!c_idx); } static int explicit_rdpcm_dir_flag_decode(HEVCLocalContext *lc, int c_idx) { - return GET_CABAC(elem_offset[EXPLICIT_RDPCM_DIR_FLAG] + !!c_idx); + return GET_CABAC(EXPLICIT_RDPCM_DIR_FLAG_OFFSET + !!c_idx); } int ff_hevc_log2_res_scale_abs(HEVCLocalContext *lc, int idx) { int i =0; - while (i < 4 && GET_CABAC(elem_offset[LOG2_RES_SCALE_ABS] + 4 * idx + i)) + while (i < 4 && GET_CABAC(LOG2_RES_SCALE_ABS_OFFSET + 4 * idx + i)) i++; return i; @@ -918,7 +870,7 @@ int ff_hevc_log2_res_scale_abs(HEVCLocalContext *lc, int idx) int ff_hevc_res_scale_sign_flag(HEVCLocalContext *lc, int idx) { - return GET_CABAC(elem_offset[RES_SCALE_SIGN_FLAG] + idx); + return GET_CABAC(RES_SCALE_SIGN_FLAG_OFFSET + idx); } static av_always_inline void last_significant_coeff_xy_prefix_decode(HEVCLocalContext *lc, int c_idx, @@ -936,13 +888,13 @@ static av_always_inline void last_significant_coeff_xy_prefix_decode(HEVCLocalCo ctx_shift = log2_size - 2; } while (i < max && - GET_CABAC(elem_offset[LAST_SIGNIFICANT_COEFF_X_PREFIX] + (i >> ctx_shift) + ctx_offset)) + GET_CABAC(LAST_SIGNIFICANT_COEFF_X_PREFIX_OFFSET + (i >> ctx_shift) + ctx_offset)) i++; *last_scx_prefix = i; i = 0; while (i < max && - GET_CABAC(elem_offset[LAST_SIGNIFICANT_COEFF_Y_PREFIX] + (i >> ctx_shift) + ctx_offset)) + GET_CABAC(LAST_SIGNIFICANT_COEFF_Y_PREFIX_OFFSET + (i >> ctx_shift) + ctx_offset)) i++; *last_scy_prefix = i; } @@ -965,18 +917,18 @@ static av_always_inline int significant_coeff_group_flag_decode(HEVCLocalContext inc = FFMIN(ctx_cg, 1) + (c_idx>0 ? 2 : 0); - return GET_CABAC(elem_offset[SIGNIFICANT_COEFF_GROUP_FLAG] + inc); + return GET_CABAC(SIGNIFICANT_COEFF_GROUP_FLAG_OFFSET + inc); } static av_always_inline int significant_coeff_flag_decode(HEVCLocalContext *lc, int x_c, int y_c, int offset, const uint8_t *ctx_idx_map) { int inc = ctx_idx_map[(y_c << 2) + x_c] + offset; - return GET_CABAC(elem_offset[SIGNIFICANT_COEFF_FLAG] + inc); + return GET_CABAC(SIGNIFICANT_COEFF_FLAG_OFFSET + inc); } static av_always_inline int significant_coeff_flag_decode_0(HEVCLocalContext *lc, int c_idx, int offset) { - return GET_CABAC(elem_offset[SIGNIFICANT_COEFF_FLAG] + offset); + return GET_CABAC(SIGNIFICANT_COEFF_FLAG_OFFSET + offset); } static av_always_inline int coeff_abs_level_greater1_flag_decode(HEVCLocalContext *lc, int c_idx, int inc) @@ -985,7 +937,7 @@ static av_always_inline int coeff_abs_level_greater1_flag_decode(HEVCLocalContex if (c_idx > 0) inc += 16; - return GET_CABAC(elem_offset[COEFF_ABS_LEVEL_GREATER1_FLAG] + inc); + return GET_CABAC(COEFF_ABS_LEVEL_GREATER1_FLAG_OFFSET + inc); } static av_always_inline int coeff_abs_level_greater2_flag_decode(HEVCLocalContext *lc, int c_idx, int inc) @@ -993,7 +945,7 @@ static av_always_inline int coeff_abs_level_greater2_flag_decode(HEVCLocalContex if (c_idx > 0) inc += 4; - return GET_CABAC(elem_offset[COEFF_ABS_LEVEL_GREATER2_FLAG] + inc); + return GET_CABAC(COEFF_ABS_LEVEL_GREATER2_FLAG_OFFSET + inc); } static av_always_inline int coeff_abs_level_remaining_decode(HEVCLocalContext *lc, int rc_rice_param) diff --git a/libavcodec/hevcdec.h b/libavcodec/hevcdec.h index c697119891..e65a6180ca 100644 --- a/libavcodec/hevcdec.h +++ b/libavcodec/hevcdec.h @@ -86,58 +86,6 @@ enum RPSType { NB_RPS_TYPE, }; -enum SyntaxElement { - SAO_MERGE_FLAG = 0, - SAO_TYPE_IDX, - SAO_EO_CLASS, - SAO_BAND_POSITION, - SAO_OFFSET_ABS, - SAO_OFFSET_SIGN, - END_OF_SLICE_FLAG, - SPLIT_CODING_UNIT_FLAG, - CU_TRANSQUANT_BYPASS_FLAG, - SKIP_FLAG, - CU_QP_DELTA, - PRED_MODE_FLAG, - PART_MODE, - PCM_FLAG, - PREV_INTRA_LUMA_PRED_FLAG, - MPM_IDX, - REM_INTRA_LUMA_PRED_MODE, - INTRA_CHROMA_PRED_MODE, - MERGE_FLAG, - MERGE_IDX, - INTER_PRED_IDC, - REF_IDX_L0, - REF_IDX_L1, - ABS_MVD_GREATER0_FLAG, - ABS_MVD_GREATER1_FLAG, - ABS_MVD_MINUS2, - MVD_SIGN_FLAG, - MVP_LX_FLAG, - NO_RESIDUAL_DATA_FLAG, - SPLIT_TRANSFORM_FLAG, - CBF_LUMA, - CBF_CB_CR, - TRANSFORM_SKIP_FLAG, - EXPLICIT_RDPCM_FLAG, - EXPLICIT_RDPCM_DIR_FLAG, - LAST_SIGNIFICANT_COEFF_X_PREFIX, - LAST_SIGNIFICANT_COEFF_Y_PREFIX, - LAST_SIGNIFICANT_COEFF_X_SUFFIX, - LAST_SIGNIFICANT_COEFF_Y_SUFFIX, - SIGNIFICANT_COEFF_GROUP_FLAG, - SIGNIFICANT_COEFF_FLAG, - COEFF_ABS_LEVEL_GREATER1_FLAG, - COEFF_ABS_LEVEL_GREATER2_FLAG, - COEFF_ABS_LEVEL_REMAINING, - COEFF_SIGN_FLAG, - LOG2_RES_SCALE_ABS, - RES_SCALE_SIGN_FLAG, - CU_CHROMA_QP_OFFSET_FLAG, - CU_CHROMA_QP_OFFSET_IDX, -}; - enum PartMode { PART_2Nx2N = 0, PART_2NxN = 1,
This is easily possible with an X macro. Using an enum for the offsets also allows to remove two arrays which are not really needed and will typically be optimized away by the compiler: The first just exists to count the number of syntax elements*, the second one exists to get offset[CONSTANT]. These constants were of type enum SyntaxElement and this enum was only used in hevc_cabac.c (although it was declared in hevcdec.h); it is now no longer needed at all and has therefore been removed. The first of these arrays led to a warning from Clang which is fixed by this commit: warning: variable 'num_bins_in_se' is not needed and will not be emitted [-Wunneeded-internal-declaration] *: One could also just added a trailing SYNTAX_ELEMENT_NB to the SyntaxElement enum for this purpose. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com> --- libavcodec/hevc_cabac.c | 248 ++++++++++++++++------------------------ libavcodec/hevcdec.h | 52 --------- 2 files changed, 100 insertions(+), 200 deletions(-)