diff options
author | Frank Barchard <fbarchard@google.com> | 2017-11-08 15:41:07 -0800 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2017-11-09 00:51:06 +0000 |
commit | 735ace2ed3389d590e35060dfb17b331568f78fd (patch) | |
tree | c2e13f23f30d9538ecc5cfbacdc56793efb744bf /include/libyuv/planar_functions.h | |
parent | 68f852d835c5da218080f1e04256b0fbc6c21e9c (diff) | |
download | libyuv-735ace2ed3389d590e35060dfb17b331568f78fd.tar.gz |
Re-enable x86 assembly without requiring -msse2
clang does not require -msse2 or -msse for inline, except
the "x" parameter. So change this to "m" for 32 bit. 64 bit
requires sse2 so use "x" for 64 bit.
gcc requires -msse for xmm registers in clobber list.
Reduce compiler requirement from -msse2 to -msse for enabling
assembly.
Bug: libyuv:754, libyuv:757
Test: CC=clang CXX=clang++ CFLAGS="-m32" CXXFLAGS="-m32 -mno-sse -O2" make -f linux.mk
Change-Id: I86df72cfee80b7d349561c1fd7c97ad360767255
Reviewed-on: https://chromium-review.googlesource.com/759303
Reviewed-by: richard winterton <rrwinterton@gmail.com>
Reviewed-by: Frank Barchard <fbarchard@google.com>
Commit-Queue: Frank Barchard <fbarchard@google.com>
Diffstat (limited to 'include/libyuv/planar_functions.h')
-rw-r--r-- | include/libyuv/planar_functions.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/libyuv/planar_functions.h b/include/libyuv/planar_functions.h index d97965cb..c91501a9 100644 --- a/include/libyuv/planar_functions.h +++ b/include/libyuv/planar_functions.h @@ -746,7 +746,7 @@ int I420Interpolate(const uint8* src0_y, int interpolation); #if defined(__pnacl__) || defined(__CLR_VER) || \ - (defined(__i386__) && !defined(__SSE2__)) + (defined(__i386__) && !defined(__SSE__) && !defined(__clang__)) #define LIBYUV_DISABLE_X86 #endif // MemorySanitizer does not support assembly code yet. http://crbug.com/344505 |