From 672100a84e11a4ce61edcfa080f994d887a5d8b3 Mon Sep 17 00:00:00 2001 From: Johann Date: Tue, 21 Feb 2017 11:12:45 -0800 Subject: [PATCH] vp8_fdct4x4 test: fix segfault again 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 | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/test/vp8_fdct4x4_test.cc b/test/vp8_fdct4x4_test.cc index 4b3282da5..9f69ae164 100644 --- a/test/vp8_fdct4x4_test.cc +++ b/test/vp8_fdct4x4_test.cc @@ -86,7 +86,7 @@ class FdctTest : public ::testing::TestWithParam { 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) { -- 2.50.1