commit 9f02130ad64dbb9f6d7d1280a6b9a317fe8a1c51
author Tom Finegan <tomfinegan@google.com> Thu Sep 07 07:49:42 2017 -0700
committer James Zern <jzern@google.com> Fri Sep 08 06:01:49 2017 +0000
tree 8395a34df98314044689fd132263a492a544de5a
parent 3748bc2aee8955bcb41b7c115c02825ba6494edc

Fix warning in selfguided_filter_test.cc.

When building without SSE4 support some compilers complain about
an unused variable. Typically gcc's with a major version of 4.
Guard the declaration of the offending var within a
HAVE_SSE4_1 block to avoid the problem.

Change-Id: I4e4deb46014c97f3157f3b6c2376e1b034a51b62

test/selfguided_filter_test.cc

diff --git a/test/selfguided_filter_test.cc b/test/selfguided_filter_test.cc
index 736e3f4..eaf5520 100644
--- a/test/selfguided_filter_test.cc
+++ b/test/selfguided_filter_test.cc
@@ -135,9 +135,8 @@
 TEST_P(AV1SelfguidedFilterTest, SpeedTest) { RunSpeedTest(); }
 TEST_P(AV1SelfguidedFilterTest, CorrectnessTest) { RunCorrectnessTest(); }
 
-const FilterTestParam params[] = { make_tuple() };
-
 #if HAVE_SSE4_1
+const FilterTestParam params[] = { make_tuple() };
 INSTANTIATE_TEST_CASE_P(SSE4_1, AV1SelfguidedFilterTest,
                         ::testing::ValuesIn(params));
 #endif
@@ -258,10 +257,9 @@
 TEST_P(AV1HighbdSelfguidedFilterTest, SpeedTest) { RunSpeedTest(); }
 TEST_P(AV1HighbdSelfguidedFilterTest, CorrectnessTest) { RunCorrectnessTest(); }
 
+#if HAVE_SSE4_1
 const HighbdFilterTestParam highbd_params[] = { make_tuple(8), make_tuple(10),
                                                 make_tuple(12) };
-
-#if HAVE_SSE4_1
 INSTANTIATE_TEST_CASE_P(SSE4_1, AV1HighbdSelfguidedFilterTest,
                         ::testing::ValuesIn(highbd_params));
 #endif