From: Johann Date: Tue, 21 Feb 2017 19:12:45 +0000 (-0800) Subject: vp8_fdct4x4 test: fix segfault again X-Git-Tag: v1.7.0~679^2 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=672100a84e11a4ce61edcfa080f994d887a5d8b3;p=libvpx 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 --- 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) {