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

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

Issue 1658903002: Enable /analyze in gn builds (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: gn format, and manual tweaks Created 4 years, 10 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 | « no previous file | build/config/win/BUILD.gn » ('j') | build/config/win/BUILD.gn » ('J')
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 # PLATFORM SELECTION 6 # PLATFORM SELECTION
7 # ============================================================================= 7 # =============================================================================
8 # 8 #
9 # There are two main things to set: "os" and "cpu". The "toolchain" is the name 9 # There are two main things to set: "os" and "cpu". The "toolchain" is the name
10 # of the GN thing that encodes combinations of these things. 10 # of the GN thing that encodes combinations of these things.
(...skipping 448 matching lines...) Expand 10 before | Expand all | Expand 10 after
459 "//build/config/compiler:runtime_library", 459 "//build/config/compiler:runtime_library",
460 "//build/config/sanitizers:default_sanitizer_flags", 460 "//build/config/sanitizers:default_sanitizer_flags",
461 "//build/config/sanitizers:default_sanitizer_coverage_flags", 461 "//build/config/sanitizers:default_sanitizer_coverage_flags",
462 ] 462 ]
463 if (is_win) { 463 if (is_win) {
464 _native_compiler_configs += [ 464 _native_compiler_configs += [
465 "//build/config/win:lean_and_mean", 465 "//build/config/win:lean_and_mean",
466 "//build/config/win:nominmax", 466 "//build/config/win:nominmax",
467 "//build/config/win:unicode", 467 "//build/config/win:unicode",
468 "//build/config/win:winver", 468 "//build/config/win:winver",
469 "//build/config/win:win_analyze",
brettw 2016/02/03 18:48:00 The only time we need to add stuff in this file is
brucedawson 2016/02/06 01:07:30 Some targets do indeed need to opt-out of this con
469 ] 470 ]
470 } 471 }
471 if (current_os == "winrt_81" || current_os == "winrt_81_phone" || 472 if (current_os == "winrt_81" || current_os == "winrt_81_phone" ||
472 current_os == "winrt_10") { 473 current_os == "winrt_10") {
473 _native_compiler_configs += [ "//build/config/win:target_winrt" ] 474 _native_compiler_configs += [ "//build/config/win:target_winrt" ]
474 } 475 }
475 if (is_posix) { 476 if (is_posix) {
476 _native_compiler_configs += [ 477 _native_compiler_configs += [
477 "//build/config/gcc:no_exceptions", 478 "//build/config/gcc:no_exceptions",
478 "//build/config/gcc:symbol_visibility_hidden", 479 "//build/config/gcc:symbol_visibility_hidden",
(...skipping 123 matching lines...) Expand 10 before | Expand all | Expand 10 after
602 forward_variables_from(invoker, "*") 603 forward_variables_from(invoker, "*")
603 604
604 # All shared libraries must have the sanitizer deps to properly link in 605 # All shared libraries must have the sanitizer deps to properly link in
605 # asan mode (this target will be empty in other cases). 606 # asan mode (this target will be empty in other cases).
606 if (!defined(deps)) { 607 if (!defined(deps)) {
607 deps = [] 608 deps = []
608 } 609 }
609 deps += [ "//build/config/sanitizers:deps" ] 610 deps += [ "//build/config/sanitizers:deps" ]
610 } 611 }
611 } 612 }
OLDNEW
« no previous file with comments | « no previous file | build/config/win/BUILD.gn » ('j') | build/config/win/BUILD.gn » ('J')

Powered by Google App Engine
This is Rietveld 408576698