Refactor av1_build_inter_predictors_sb().
Before this, this function was copied-and-edited from
av1_build_inter_predictors_sby() and av1_build_inter_predictors_sbuv().
Also, some places used sb() one directly (like rdopt.c), while others used
sby() and sbuv() separately (like encodeframe.c --> encode_superblock()).
This was prone to errors, if someone mistakenly modified only one/two of
the 3 functions.
Now, we just reuse sby() and sbuv() functions in sb().
Verified that there's no change in output.
Change-Id: If81d78e767cc3fa09f06094d4db36ff1eaea3e4f
diff --git a/av1/encoder/encodeframe.c b/av1/encoder/encodeframe.c
index 8a65d88..5e7975a 100644
--- a/av1/encoder/encodeframe.c
+++ b/av1/encoder/encodeframe.c
@@ -6020,8 +6020,7 @@
}
#endif // CONFIG_EXT_INTER && CONFIG_COMPOUND_SINGLEREF
- av1_build_inter_predictors_sby(cm, xd, mi_row, mi_col, NULL, block_size);
- av1_build_inter_predictors_sbuv(cm, xd, mi_row, mi_col, NULL, block_size);
+ av1_build_inter_predictors_sb(cm, xd, mi_row, mi_col, NULL, block_size);
#if CONFIG_MOTION_VAR
if (mbmi->motion_mode == OBMC_CAUSAL) {
#if CONFIG_NCOBMC