From: James Zern <jzern@google.com>
Date: Thu, 9 Dec 2021 05:35:26 +0000 (-0800)
Subject: test_intra_pred_speed: match above ext w/reconintra
X-Git-Tag: v1.12.0-rc1~111^2
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=093a8c4824729be62e582b7d3f00e18830aee3b8;p=platform%2Fupstream%2Flibvpx.git

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
---

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];
     }
   }