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

Side by Side Diff: build/config/compiler/BUILD.gn

Issue 1390223002: Enforce WARN_UNUSED_RESULT attribute on Windows (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 2 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
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 import("//build/config/android/config.gni") 5 import("//build/config/android/config.gni")
6 import("//build/config/chrome_build.gni") 6 import("//build/config/chrome_build.gni")
7 import("//build/config/compiler/compiler.gni") 7 import("//build/config/compiler/compiler.gni")
8 import("//build/toolchain/ccache.gni") 8 import("//build/toolchain/ccache.gni")
9 9
10 if (current_cpu == "arm") { 10 if (current_cpu == "arm") {
(...skipping 558 matching lines...) Expand 10 before | Expand all | Expand 10 after
569 # Building with Clang on Windows is a work in progress and very 569 # Building with Clang on Windows is a work in progress and very
570 # experimental. See crbug.com/82385. 570 # experimental. See crbug.com/82385.
571 # Keep this in sync with the similar block in build/common.gypi 571 # Keep this in sync with the similar block in build/common.gypi
572 if (is_clang) { 572 if (is_clang) {
573 cflags += [ 573 cflags += [
574 # TODO(hans): Make this list shorter eventually, http://crbug.com/504657 574 # TODO(hans): Make this list shorter eventually, http://crbug.com/504657
575 "-Qunused-arguments", # http://crbug.com/504658 575 "-Qunused-arguments", # http://crbug.com/504658
576 "-Wno-microsoft-enum-value", # http://crbug.com/505296 576 "-Wno-microsoft-enum-value", # http://crbug.com/505296
577 "-Wno-unknown-pragmas", # http://crbug.com/505314 577 "-Wno-unknown-pragmas", # http://crbug.com/505314
578 "-Wno-unused-value", # http://crbug.com/505318 578 "-Wno-unused-value", # http://crbug.com/505318
579 "-Wunused-result",
hans 2015/10/07 16:25:26 (See the common.gypi comment.)
579 ] 580 ]
580 } 581 }
581 } else { 582 } else {
582 # Common GCC warning setup. 583 # Common GCC warning setup.
583 cflags += [ 584 cflags += [
584 # Enables. 585 # Enables.
585 "-Wendif-labels", # Weird old-style text after an #endif. 586 "-Wendif-labels", # Weird old-style text after an #endif.
586 "-Werror", # Warnings as errors. 587 "-Werror", # Warnings as errors.
587 588
588 # Disables. 589 # Disables.
(...skipping 524 matching lines...) Expand 10 before | Expand all | Expand 10 after
1113 if (symbol_level == 0) { 1114 if (symbol_level == 0) {
1114 configs = [ ":no_symbols" ] 1115 configs = [ ":no_symbols" ]
1115 } else if (symbol_level == 1) { 1116 } else if (symbol_level == 1) {
1116 configs = [ ":minimal_symbols" ] 1117 configs = [ ":minimal_symbols" ]
1117 } else if (symbol_level == 2) { 1118 } else if (symbol_level == 2) {
1118 configs = [ ":symbols" ] 1119 configs = [ ":symbols" ]
1119 } else { 1120 } else {
1120 assert(false) 1121 assert(false)
1121 } 1122 }
1122 } 1123 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698