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

Unified Diff: build/config/BUILDCONFIG.gn

Issue 119803002: Add GN conversions for more GYP flags (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years 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 | build/gyp_chromium » ('j') | build/gyp_chromium » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/config/BUILDCONFIG.gn
diff --git a/build/config/BUILDCONFIG.gn b/build/config/BUILDCONFIG.gn
index 9e5f1094f7b59ba13401b186fd9438a0e4ae8655..dd1d2764896a719fb3a6d55089263823aa8c56b4 100644
--- a/build/config/BUILDCONFIG.gn
+++ b/build/config/BUILDCONFIG.gn
@@ -65,6 +65,18 @@ declare_args() {
# true means official Google Chrome branding (requires extra Google-internal
# resources).
is_chrome_branded = false
+
+ # Compile for Address Sanitizer to find memory bugs.
+ is_asan = false
+
+ # Compile for Leak Sanitizer to find leaks.
+ is_lsan = false
+
+ # Compile for Memory Sanitizer to find uninitialized reads.
+ is_msan = false
+
+ # Compile for Thread Sanitizer to find threading bugs.
+ is_tsan = false
}
# =============================================================================
@@ -280,6 +292,12 @@ if (is_component_build) {
component_mode = "static_library"
}
+# These Sanitizers all imply using the Clang compiler. On Windows they either
+# don't work or work differently.
+if (!is_clang && (is_asan || is_lsan || is_tsan || is_msan)) {
+ is_clang = true
+}
+
toolkit_uses_gtk = is_linux
# =============================================================================
@@ -422,7 +440,7 @@ if (is_win) {
} else if (build_cpu_arch == "x64") {
set_default_toolchain("//build/toolchain/linux:64")
}
-} else if (is_mac) {
+} else if (is_mac || is_ios) {
host_toolchain = "//build/toolchain/mac:clang"
set_default_toolchain(host_toolchain)
}
« no previous file with comments | « no previous file | build/gyp_chromium » ('j') | build/gyp_chromium » ('J')

Powered by Google App Engine
This is Rietveld 408576698