From d4cfc7bca1e6d66431b98b78dc2e29d9ec1bf904 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Martin=20Storsj=C3=B6?= Date: Fri, 30 Mar 2018 23:39:20 +0300 Subject: [PATCH] Fix building with clang targeting x86_32 mingw In these cases, CLANG_HAS_AVX2 is defined, but _MSC_VER isn't. The corresponding codepaths are in source/compare_win.cc, and the whole file is guarded with a defined(_MSC_VER). This is the same fix as in d8680893ecfcffd952c7a5d0dc1878d1c486a4e4, but this issue wasn't noticed since compare.o wasn't linked in in earlier tested configurations. Change-Id: Ifff6d4bd7b1071256df1169881490a52026c183b Reviewed-on: https://chromium-review.googlesource.com/988573 Reviewed-by: Frank Barchard Commit-Queue: Frank Barchard --- include/libyuv/compare_row.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/libyuv/compare_row.h b/include/libyuv/compare_row.h index 72ee74060..d38f9987d 100644 --- a/include/libyuv/compare_row.h +++ b/include/libyuv/compare_row.h @@ -46,7 +46,7 @@ extern "C" { #endif // __clang__ // The following are available for Visual C: -#if !defined(LIBYUV_DISABLE_X86) && defined(_M_IX86) && \ +#if !defined(LIBYUV_DISABLE_X86) && defined(_M_IX86) && defined(_MSC_VER) && \ (defined(VISUALC_HAS_AVX2) || defined(CLANG_HAS_AVX2)) #define HAS_HASHDJB2_AVX2 #endif