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/scale_row.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/scale_row.h')
| -rw-r--r-- | include/libyuv/scale_row.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/libyuv/scale_row.h b/include/libyuv/scale_row.h index 534898cd..c4a66aa0 100644 --- a/include/libyuv/scale_row.h +++ b/include/libyuv/scale_row.h @@ -20,7 +20,7 @@ extern "C" { #endif #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 |