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

Side by Side Diff: build/config/BUILDCONFIG.gn

Issue 1134003008: GN: Implement is_lsan, is_tsan and is_msan. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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 unified diff | Download patch
« no previous file with comments | « build/config/BUILD.gn ('k') | build/config/allocator.gni » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright (c) 2013 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2013 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 # ============================================================================= 5 # =============================================================================
6 # BUILD FLAGS 6 # BUILD FLAGS
7 # ============================================================================= 7 # =============================================================================
8 # 8 #
9 # This block lists input arguments to the build, along with their default 9 # This block lists input arguments to the build, along with their default
10 # values. GN requires listing them explicitly so it can validate input and have 10 # values. GN requires listing them explicitly so it can validate input and have
(...skipping 59 matching lines...) Expand 10 before | Expand all | Expand 10 after
70 is_official_build = false 70 is_official_build = false
71 71
72 # Select the desired branding flavor. False means normal Chromium branding, 72 # Select the desired branding flavor. False means normal Chromium branding,
73 # true means official Google Chrome branding (requires extra Google-internal 73 # true means official Google Chrome branding (requires extra Google-internal
74 # resources). 74 # resources).
75 # TODO(brettw) move to chrome_build.gni when DEPS are updated. 75 # TODO(brettw) move to chrome_build.gni when DEPS are updated.
76 is_chrome_branded = false 76 is_chrome_branded = false
77 77
78 # Compile for Address Sanitizer to find memory bugs. 78 # Compile for Address Sanitizer to find memory bugs.
79 is_asan = false 79 is_asan = false
80 asan_blacklist = "//tools/memory/asan/blacklist.txt"
brettw 2015/05/19 22:54:27 Please don't add any more args to this file. I'm a
Sam McNally 2015/05/20 01:07:21 I think it's clear for args used by a subset of th
80 81
81 # Compile for Leak Sanitizer to find leaks. 82 # Compile for Leak Sanitizer to find leaks.
82 is_lsan = false 83 is_lsan = false
83 84
84 # Compile for Memory Sanitizer to find uninitialized reads. 85 # Compile for Memory Sanitizer to find uninitialized reads.
85 is_msan = false 86 is_msan = false
87 msan_blacklist = "//tools/msan/blacklist.txt"
88
89 # Track where uninitialized memory originates from. From fastest to slowest:
90 # 0 - no tracking, 1 - track only the initial allocation site, 2 - track the
91 # chain of stores leading from allocation site to use site.
92 msan_track_origins = 2
86 93
87 # Compile for Thread Sanitizer to find threading bugs. 94 # Compile for Thread Sanitizer to find threading bugs.
88 is_tsan = false 95 is_tsan = false
96 tsan_blacklist = "//tools/memory/tsan_v2/ignores.txt"
89 97
90 if (current_os == "chromeos") { 98 if (current_os == "chromeos") {
91 # Allows the target toolchain to be injected as arguments. This is needed 99 # Allows the target toolchain to be injected as arguments. This is needed
92 # to support the CrOS build system which supports per-build-configuration 100 # to support the CrOS build system which supports per-build-configuration
93 # toolchains. 101 # toolchains.
94 cros_use_custom_toolchain = false 102 cros_use_custom_toolchain = false
95 } 103 }
96 } 104 }
97 105
98 # ============================================================================= 106 # =============================================================================
(...skipping 267 matching lines...) Expand 10 before | Expand all | Expand 10 after
366 _default_optimization_config = "//build/config/compiler:optimize" 374 _default_optimization_config = "//build/config/compiler:optimize"
367 } 375 }
368 _native_compiler_configs += [ _default_optimization_config ] 376 _native_compiler_configs += [ _default_optimization_config ]
369 377
370 # If it wasn't manually set, set to an appropriate default. 378 # If it wasn't manually set, set to an appropriate default.
371 if (symbol_level == -1) { 379 if (symbol_level == -1) {
372 # Linux is slowed by having symbols as part of the target binary, whereas 380 # Linux is slowed by having symbols as part of the target binary, whereas
373 # Mac and Windows have them separate, so in Release Linux, default them off. 381 # Mac and Windows have them separate, so in Release Linux, default them off.
374 if (is_debug || !is_linux) { 382 if (is_debug || !is_linux) {
375 symbol_level = 2 383 symbol_level = 2
384 } else if (is_asan || is_lsan || is_tsan || is_msan) {
385 # Sanitizers require symbols for filename suppressions to work.
386 symbol_level = 1
376 } else { 387 } else {
377 symbol_level = 0 388 symbol_level = 0
378 } 389 }
379 } 390 }
380 391
381 # Symbol setup. 392 # Symbol setup.
382 if (symbol_level == 2) { 393 if (symbol_level == 2) {
383 _default_symbols_config = "//build/config/compiler:symbols" 394 _default_symbols_config = "//build/config/compiler:symbols"
384 } else if (symbol_level == 1) { 395 } else if (symbol_level == 1) {
385 _default_symbols_config = "//build/config/compiler:minimal_symbols" 396 _default_symbols_config = "//build/config/compiler:minimal_symbols"
(...skipping 340 matching lines...) Expand 10 before | Expand all | Expand 10 after
726 } 737 }
727 if (defined(invoker.testonly)) { 738 if (defined(invoker.testonly)) {
728 testonly = invoker.testonly 739 testonly = invoker.testonly
729 } 740 }
730 if (defined(invoker.visibility)) { 741 if (defined(invoker.visibility)) {
731 visibility = invoker.visibility 742 visibility = invoker.visibility
732 } 743 }
733 } 744 }
734 } 745 }
735 } 746 }
OLDNEW
« no previous file with comments | « build/config/BUILD.gn ('k') | build/config/allocator.gni » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698