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

Unified Diff: build/config/compiler/BUILD.gn

Issue 1952043003: Add SSE2/MMX build flags when building on an x86 host. This is necessary (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 7 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: build/config/compiler/BUILD.gn
diff --git a/build/config/compiler/BUILD.gn b/build/config/compiler/BUILD.gn
index 2f48477b9750977c999af1c6408cdb762ebba269..a608b4a9453c237971e1edaa61b1ecb9bfaf4868 100644
--- a/build/config/compiler/BUILD.gn
+++ b/build/config/compiler/BUILD.gn
@@ -219,6 +219,14 @@ config("compiler") {
}
}
+ if (current_cpu == "x86") {
+ cflags += [
+ "-msse2",
+ "-mfpmath=sse",
+ "-mmmx",
Nico 2016/05/05 17:33:23 doesn't msse2 imply mmx?
saiarcot895 2016/05/05 21:25:18 It might (based on the options for the -march flag
+ ]
+ }
+
if (is_clang && is_debug) {
# Allow comparing the address of references and 'this' against 0
# in debug builds. Technically, these can never be null in
« 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