diff options
author | Lei Zhang <thestig@chromium.org> | 2015-12-16 19:07:14 -0800 |
---|---|---|
committer | Lei Zhang <thestig@chromium.org> | 2015-12-16 19:07:14 -0800 |
commit | 692c03ffe2bd7033dc2e31163095efebace7e642 (patch) | |
tree | 94d7662816e01fee2092f7f333b51294b3183533 /build | |
parent | 301832eb1ae344d460f494ec7704d00df19bd526 (diff) | |
download | pdfium-692c03ffe2bd7033dc2e31163095efebace7e642.tar.xz |
Merge to XFA: Make clang_warning_flags work for standalone builds.
Also fix a newly introduced override warning.
Also define a host_clang GYP variable.
TBR=thakis@chromium.org
Review URL: https://codereview.chromium.org/1532723003 .
Review URL: https://codereview.chromium.org/1533763002 .
(cherry picked from commit e096ca507db9944aebc47d6c2cc1c6fab39498c5)
(cherry picked from commit 0a9158b99a2002fb82301ebec20dbc23b3fc084c)
Review URL: https://codereview.chromium.org/1535603003 .
Diffstat (limited to 'build')
-rw-r--r-- | build/set_clang_warning_flags.gypi | 58 | ||||
-rw-r--r-- | build/standalone.gypi | 6 |
2 files changed, 64 insertions, 0 deletions
diff --git a/build/set_clang_warning_flags.gypi b/build/set_clang_warning_flags.gypi new file mode 100644 index 0000000000..f6d7aea700 --- /dev/null +++ b/build/set_clang_warning_flags.gypi @@ -0,0 +1,58 @@ +# Copyright (c) 2014 The Chromium Authors. All rights reserved. +# Use of this source code is governed by a BSD-style license that can be +# found in the LICENSE file. + +# This file is meant to be included to set clang-specific compiler flags. +# To use this the following variable can be defined: +# clang_warning_flags: list: Compiler flags to pass to clang. +# clang_warning_flags_unset: list: Compiler flags to not pass to clang. +# +# Only use this in third-party code. In chromium_code, fix your code to not +# warn instead! +# +# Note that the gypi file is included in target_defaults, so it does not need +# to be explicitly included. +# +# Warning flags set by this will be used on all platforms. If you want to set +# warning flags on only some platforms, you have to do so manually. +# +# To use this, create a gyp target with the following form: +# { +# 'target_name': 'my_target', +# 'variables': { +# 'clang_warning_flags': ['-Wno-awesome-warning'], +# 'clang_warning_flags_unset': ['-Wpreviously-set-flag'], +# } +# } + +{ + 'variables': { + 'clang_warning_flags_unset%': [], # Provide a default value. + }, + 'conditions': [ + ['clang==1', { + # This uses >@ instead of @< to also see clang_warning_flags set in + # targets directly, not just the clang_warning_flags in target_defaults. + 'cflags': [ '>@(clang_warning_flags)' ], + 'cflags!': [ '>@(clang_warning_flags_unset)' ], + 'xcode_settings': { + 'WARNING_CFLAGS': ['>@(clang_warning_flags)'], + 'WARNING_CFLAGS!': ['>@(clang_warning_flags_unset)'], + }, + 'msvs_settings': { + 'VCCLCompilerTool': { + 'AdditionalOptions': [ '>@(clang_warning_flags)' ], + 'AdditionalOptions!': [ '>@(clang_warning_flags_unset)' ], + }, + }, + }], + ['clang==0 and host_clang==1', { + 'target_conditions': [ + ['_toolset=="host"', { + 'cflags': [ '>@(clang_warning_flags)' ], + 'cflags!': [ '>@(clang_warning_flags_unset)' ], + }], + ], + }], + ], +} diff --git a/build/standalone.gypi b/build/standalone.gypi index 2eac19882c..de638de97c 100644 --- a/build/standalone.gypi +++ b/build/standalone.gypi @@ -59,8 +59,10 @@ }], ['OS=="linux" or OS=="mac"', { 'clang%': 1, + 'host_clang%': 1, }, { 'clang%': 0, + 'host_clang%': 0, }], # Set default gomadir. ['OS=="win"', { @@ -261,6 +263,10 @@ '-Wno-unused-parameter', ], }, + 'variables': { + 'clang_warning_flags': [], + }, + 'includes': [ 'set_clang_warning_flags.gypi', ], 'conditions': [ ['component=="shared_library"', { 'cflags': [ |