From fbf5681aae26930cbf1ba4b4ba17b2d1cb11f98d Mon Sep 17 00:00:00 2001 From: Yi Luo Date: Fri, 2 Sep 2016 11:46:46 -0700 Subject: [PATCH] Add a TODO for aom_highbd_fdct16x16_1_sse2 tests - 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 | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/test/dct16x16_test.cc b/test/dct16x16_test.cc index 42fe6996f..227e2e204 100644 --- a/test/dct16x16_test.cc +++ b/test/dct16x16_test.cc @@ -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))); -- 2.50.1