vp8_fdct4x4 test: fix segfault again
authorJohann <johannkoenig@google.com>
Tue, 21 Feb 2017 19:12:45 +0000 (11:12 -0800)
committerJohann Koenig <johannkoenig@google.com>
Wed, 22 Feb 2017 18:29:11 +0000 (18:29 +0000)
The output needs to be aligned. Input is read with 'movq' not 'movqda'
so it is not expected to be aligned.

Change-Id: Ibd48a84c1785917a6a97c3689a05322abba486b4

test/vp8_fdct4x4_test.cc

index 4b3282d..9f69ae1 100644 (file)
@@ -86,7 +86,7 @@ class FdctTest : public ::testing::TestWithParam<FdctFunc> {
 
 TEST_P(FdctTest, SignBiasCheck) {
   int16_t test_input_block[16];
-  int16_t test_output_block[16];
+  DECLARE_ALIGNED(16, int16_t, test_output_block[16]);
   const int pitch = 8;
   int count_sign_block[16][2];
   const int count_test_block = 1000000;
@@ -155,9 +155,9 @@ TEST_P(FdctTest, RoundTripErrorCheck) {
   double total_error = 0;
   const int count_test_block = 1000000;
   for (int i = 0; i < count_test_block; ++i) {
-    DECLARE_ALIGNED(16, int16_t, test_input_block[16]);
-    DECLARE_ALIGNED(16, int16_t, test_output_block[16]);
-    int16_t test_temp_block[16];
+    int16_t test_input_block[16];
+    int16_t test_output_block[16];
+    DECLARE_ALIGNED(16, int16_t, test_temp_block[16]);
 
     // Initialize a test block with input range [-255, 255].
     for (int j = 0; j < 16; ++j) {