generic-library/vpx
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

predict_test: align dst buffer to 16

Browse Source 
On 32 bit machines 'new' does not always appear to allocate sufficiently
aligned buffers, causing intermittent test failures.

Change-Id: I0db4fc73782012e4eef71dc0fb540e74fdbfcebe
This commit is contained in:
Johann 2016-09-21 13:31:59 -07:00
parent b6e686b1ea
commit 35ebc1cddf
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
test/predict_test.cc
View File

@ -20,6 +20,7 @@
#include "test/register_state_check.h" #include "test/register_state_check.h"
#include "test/util.h" #include "test/util.h"
#include "vpx/vpx_integer.h" #include "vpx/vpx_integer.h"
#include "vpx_mem/vpx_mem.h"
namespace { namespace {
@ -46,7 +47,8 @@ class PredictTestBase : public ::testing::TestWithParam<PredictParam> {
// memory to facilitate detecting out of bounds writes. // memory to facilitate detecting out of bounds writes.
dst_stride_ = kBorderSize + width_ + kBorderSize; dst_stride_ = kBorderSize + width_ + kBorderSize;
padded_dst_size_ = dst_stride_ * (kBorderSize + height_ + kBorderSize); padded_dst_size_ = dst_stride_ * (kBorderSize + height_ + kBorderSize);
padded_dst_ = new uint8_t[padded_dst_size_]; padded_dst_ =
reinterpret_cast<uint8_t *>(vpx_memalign(16, padded_dst_size_));
ASSERT_TRUE(padded_dst_ != NULL); ASSERT_TRUE(padded_dst_ != NULL);
dst_ = padded_dst_ + (kBorderSize * dst_stride_) + kBorderSize; dst_ = padded_dst_ + (kBorderSize * dst_stride_) + kBorderSize;
@ -61,7 +63,7 @@ class PredictTestBase : public ::testing::TestWithParam<PredictParam> {
virtual void TearDown() { virtual void TearDown() {
delete[] src_; delete[] src_;
src_ = NULL; src_ = NULL;
delete[] padded_dst_; vpx_free(padded_dst_);
padded_dst_ = NULL; padded_dst_ = NULL;
dst_ = NULL; dst_ = NULL;
delete[] dst_c_; delete[] dst_c_;