]> granicus.if.org Git - libvpx/commitdiff
Add a TODO for aom_highbd_fdct16x16_1_sse2 tests
authorYi Luo <luoyi@google.com>
Fri, 2 Sep 2016 18:46:46 +0000 (11:46 -0700)
committerYi Luo <luoyi@google.com>
Mon, 19 Sep 2016 23:26:08 +0000 (16:26 -0700)
- Here function, aom_fdct16x16_1_sse2 is mistakely tested. It can pass
  AOM_BITS_8, AOM_BITS_10, but not AOM_BITS_12. We should fix this test
  when aom_highbd_fdct16x16_1_sse2 is available.

Change-Id: I5cac6ee5404ff6d833940e1ecc34663b29d7a41c

test/dct16x16_test.cc

index 42fe6996f0692cbbf2b1022592046f89600902c9..227e2e204ce1a3401c4fc7bcfb99e69765e8413f 100644 (file)
@@ -910,6 +910,11 @@ INSTANTIATE_TEST_CASE_P(
                                  &idct16x16_10_add_12_sse2, 3167, AOM_BITS_12),
                       make_tuple(&idct16x16_12, &idct16x16_256_add_12_sse2,
                                  3167, AOM_BITS_12)));
+// TODO(luoyi):
+// For this test case, we should test function: aom_highbd_fdct16x16_1_sse2.
+// However this function is not available yet. if we mistakely test
+// aom_fdct16x16_1_sse2, it could only pass AOM_BITS_8/AOM_BITS_10 but not
+// AOM_BITS_12.
 INSTANTIATE_TEST_CASE_P(SSE2, PartialTrans16x16Test,
                         ::testing::Values(make_tuple(&aom_fdct16x16_1_sse2,
                                                      AOM_BITS_8)));