Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(593)

Unified Diff: src/opts/SkBlitRow_opts_SSE4.h

Issue 363983004: Exclude Clang on Windows too. Comment this up a bit. (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: comment Created 6 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/opts/SkBlitRow_opts_SSE4.h
diff --git a/src/opts/SkBlitRow_opts_SSE4.h b/src/opts/SkBlitRow_opts_SSE4.h
index c49c392d0acb5015012129f7edee5d2ff743c78f..f202aa5139129a72e6bc5d65c60d10874d4ce9eb 100644
--- a/src/opts/SkBlitRow_opts_SSE4.h
+++ b/src/opts/SkBlitRow_opts_SSE4.h
@@ -10,12 +10,14 @@
#include "SkBlitRow.h"
-/* Check if we are able to build assembly code, GCC/AT&T syntax.
- * Had problems with LLVM-GCC 4.2.
- * MemorySanitizer cannot handle assembly code.
+/* Check if we are able to build assembly code, GCC/AT&T syntax:
+ * 1) Clang and GCC are generally OK. OS X's old LLVM-GCC 4.2 can't handle it;
+ * 2) We're intentionally not linking this in even when supported (Clang) on Windows;
+ * 3) MemorySanitizer cannot instrument assembly at all.
*/
-#if (defined(__clang__) || (defined(__GNUC__) && !defined(SK_BUILD_FOR_MAC))) \
- && !defined(MEMORY_SANITIZER)
+#if /* 1)*/ (defined(__clang__) || (defined(__GNUC__) && !defined(SK_BUILD_FOR_MAC))) \
+ /* 2)*/ && !defined(SK_BUILD_FOR_WIN) \
+ /* 3)*/ && !defined(MEMORY_SANITIZER)
extern "C" void S32A_Opaque_BlitRow32_SSE4_asm(SkPMColor* SK_RESTRICT dst,
const SkPMColor* SK_RESTRICT src,
int count, U8CPU alpha);
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698