From 093a8c4824729be62e582b7d3f00e18830aee3b8 Mon Sep 17 00:00:00 2001 From: James Zern Date: Wed, 8 Dec 2021 21:35:26 -0800 Subject: [PATCH] test_intra_pred_speed: match above ext w/reconintra only 2 x block_size is needed + remove a related TODO; C & assembly rely on this extension Change-Id: Iea430267624251cccbbdaec8045eb81d01ae1db1 --- test/test_intra_pred_speed.cc | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/test/test_intra_pred_speed.cc b/test/test_intra_pred_speed.cc index 08100a1..28b3484 100644 --- a/test/test_intra_pred_speed.cc +++ b/test/test_intra_pred_speed.cc @@ -48,11 +48,9 @@ struct IntraPredTestMem { for (int i = 0; i < kBPS; ++i) left[i] = rnd.Rand16() & mask; for (int i = -1; i < kBPS; ++i) above[i] = rnd.Rand16() & mask; - // some code assumes the top row has been extended: - // d45/d63 C-code, for instance, but not the assembly. - // TODO(jzern): this style of extension isn't strictly necessary. + // d45/d63 require the top row to be extended. ASSERT_LE(block_size, kBPS); - for (int i = block_size; i < 2 * kBPS; ++i) { + for (int i = block_size; i < 2 * block_size; ++i) { above[i] = above[block_size - 1]; } } -- 2.7.4