shithub: libvpx

Download patch

ref: d7822442d48ead541de7d202419c63e3f8a49875
parent: 2a9698c1113e209f531fc1cbf814018ab35d1123
author: Johann <johannkoenig@google.com>
date: Wed Oct 23 06:52:38 EDT 2019

use a compile time constant for kDataAlignment

const or constexpr should be sufficient for this use but older
versions of gcc fail to expand DECLARE_ALIGNED correctly. Work
around this by using an enum.

Fixed: webm:1660
Change-Id: Ifa4f7585417760f90f9fb28332152019de9f8169

--- a/test/sad_test.cc
+++ b/test/sad_test.cc
@@ -116,7 +116,10 @@
 
  protected:
   // Handle blocks up to 4 blocks 64x64 with stride up to 128
-  static const int kDataAlignment = 32;
+  // crbug.com/webm/1660
+  // const[expr] should be sufficient for DECLARE_ALIGNED but early
+  // implementations of c++11 appear to have some issues with it.
+  enum { kDataAlignment = 32 };
   static const int kDataBlockSize = 64 * 128;
   static const int kDataBufferSize = 4 * kDataBlockSize;