libyuv: disable AVX512 in clang
authorJohann <johannkoenig@google.com>
Thu, 28 Jun 2018 13:08:03 +0000 (06:08 -0700)
committerJohann <johannkoenig@google.com>
Thu, 28 Jun 2018 20:17:42 +0000 (13:17 -0700)
ARGBToRGB24Row_AVX512VBMI fails to compile on Mac:
row_gcc.cc: instruction requires: AVX-512 VBMI ISA AVX-512 VL ISA

BUG=libyuv:789

Change-Id: Ibd584e8c82e3ce86ec5460b4243f84f5dbdf4c81

third_party/libyuv/README.libvpx
third_party/libyuv/include/libyuv/row.h

index 1216ee9..9519dc4 100644 (file)
@@ -13,6 +13,7 @@ example which down-samples the original input video (f.g. 1280x720) a number of
 times in order to encode multiple resolution bit streams.
 
 Local Modifications:
+Disable ARGBToRGB24Row_AVX512VBMI due to build failure on Mac.
 rm libyuv/include/libyuv.h libyuv/include/libyuv/compare_row.h
 mv libyuv/include tmp/
 mv libyuv/source tmp/
index 646a6ab..65ef448 100644 (file)
@@ -57,11 +57,13 @@ extern "C" {
 
 // clang >= 6.0.0 required for AVX512.
 // TODO(fbarchard): fix xcode 9 ios b/789.
+#if 0  // Build fails in libvpx on Mac
 #if defined(__clang__) && (defined(__x86_64__) || defined(__i386__))
 #if (__clang_major__ >= 7) && !defined(__APPLE_EMBEDDED_SIMULATOR__)
 #define CLANG_HAS_AVX512 1
 #endif  // clang >= 7
 #endif  // __clang__
+#endif  // 0
 
 // Visual C 2012 required for AVX2.
 #if defined(_M_IX86) && !defined(__clang__) && defined(_MSC_VER) && \