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

Side by Side Diff: build/common.gypi

Issue 781763003: Roll Clang 218707:223109 (+r223211, +r223219, -r220714) (Closed) Base URL: precise:/work/chromium/src@clang_roll_223109
Patch Set: Update shared library expectations Created 6 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 unified diff | Download patch
OLDNEW
1 # Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2012 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 # IMPORTANT: 5 # IMPORTANT:
6 # Please don't directly include this file if you are building via gyp_chromium, 6 # Please don't directly include this file if you are building via gyp_chromium,
7 # since gyp_chromium is automatically forcing its inclusion. 7 # since gyp_chromium is automatically forcing its inclusion.
8 { 8 {
9 # Variables expected to be overriden on the GYP command line (-D) or by 9 # Variables expected to be overriden on the GYP command line (-D) or by
10 # ~/.gyp/include.gypi. 10 # ~/.gyp/include.gypi.
(...skipping 2503 matching lines...) Expand 10 before | Expand all | Expand 10 after
2514 # This warns on using ints as initializers for floats in 2514 # This warns on using ints as initializers for floats in
2515 # initializer lists (e.g. |int a = f(); CGSize s = { a, a };|), 2515 # initializer lists (e.g. |int a = f(); CGSize s = { a, a };|),
2516 # which happens in several places in chrome code. Not sure if 2516 # which happens in several places in chrome code. Not sure if
2517 # this is worth fixing. 2517 # this is worth fixing.
2518 '-Wno-c++11-narrowing', 2518 '-Wno-c++11-narrowing',
2519 2519
2520 # Clang considers the `register` keyword as deprecated, but e.g. 2520 # Clang considers the `register` keyword as deprecated, but e.g.
2521 # code generated by flex (used in angle) contains that keyword. 2521 # code generated by flex (used in angle) contains that keyword.
2522 # http://crbug.com/255186 2522 # http://crbug.com/255186
2523 '-Wno-deprecated-register', 2523 '-Wno-deprecated-register',
2524
2525 # TODO(hans): Get this cleaned up.
2526 '-Wno-inconsistent-missing-override',
2524 ], 2527 ],
2525 }, 2528 },
2526 'includes': [ 'set_clang_warning_flags.gypi', ], 2529 'includes': [ 'set_clang_warning_flags.gypi', ],
2527 'defines': [ 2530 'defines': [
2528 # Don't use deprecated V8 APIs anywhere. 2531 # Don't use deprecated V8 APIs anywhere.
2529 'V8_DEPRECATION_WARNINGS', 2532 'V8_DEPRECATION_WARNINGS',
2530 ], 2533 ],
2531 'include_dirs': [ 2534 'include_dirs': [
2532 '<(SHARED_INTERMEDIATE_DIR)', 2535 '<(SHARED_INTERMEDIATE_DIR)',
2533 ], 2536 ],
(...skipping 1701 matching lines...) Expand 10 before | Expand all | Expand 10 after
4235 'defines': [ 4238 'defines': [
4236 'UNDEFINED_SANITIZER', 4239 'UNDEFINED_SANITIZER',
4237 ], 4240 ],
4238 }], 4241 }],
4239 ], 4242 ],
4240 }], 4243 }],
4241 ['asan_coverage!=0', { 4244 ['asan_coverage!=0', {
4242 'target_conditions': [ 4245 'target_conditions': [
4243 ['_toolset=="target"', { 4246 ['_toolset=="target"', {
4244 'cflags': [ 4247 'cflags': [
4245 '-mllvm -asan-coverage=<(asan_coverage)', 4248 '-fsanitize-coverage=<(asan_coverage)',
4246 ], 4249 ],
4247 }], 4250 }],
4248 ], 4251 ],
4249 }], 4252 }],
4250 ['asan_field_padding!=0', { 4253 ['asan_field_padding!=0', {
4251 'target_conditions': [ 4254 'target_conditions': [
4252 ['_toolset=="target"', { 4255 ['_toolset=="target"', {
4253 'cflags': [ 4256 'cflags': [
4254 '-fsanitize-address-field-padding=<(asan_field_padding)', 4257 '-fsanitize-address-field-padding=<(asan_field_padding)',
4255 ], 4258 ],
(...skipping 350 matching lines...) Expand 10 before | Expand all | Expand 10 after
4606 ], 4609 ],
4607 }], 4610 }],
4608 ['asan==1', { 4611 ['asan==1', {
4609 'cflags': [ 4612 'cflags': [
4610 # Android build relies on -Wl,--gc-sections removing 4613 # Android build relies on -Wl,--gc-sections removing
4611 # unreachable code. ASan instrumentation for globals inhibits 4614 # unreachable code. ASan instrumentation for globals inhibits
4612 # this and results in a library with unresolvable relocations. 4615 # this and results in a library with unresolvable relocations.
4613 # TODO(eugenis): find a way to reenable this. 4616 # TODO(eugenis): find a way to reenable this.
4614 '-mllvm -asan-globals=0', 4617 '-mllvm -asan-globals=0',
4615 ], 4618 ],
4616 'conditions': [
4617 ['target_arch=="arm"', {
4618 'ldflags': [
4619 # TODO(hans): The ASan runtime is no longer automatically
4620 # added to the link line when using -nostdlib. Can we
4621 # stop adding -nostdlib? (crbug.com/423429)
4622 '<!(cd <(DEPTH) && pwd -P)/<(make_clang_dir)/lib/clang/3.6 .0/lib/linux/libclang_rt.asan-arm-android.so',
4623 ],
4624 }],
4625 ],
4626 }], 4619 }],
4627 ['android_webview_build==0', { 4620 ['android_webview_build==0', {
4628 'defines': [ 4621 'defines': [
4629 # The NDK has these things, but doesn't define the constants 4622 # The NDK has these things, but doesn't define the constants
4630 # to say that it does. Define them here instead. 4623 # to say that it does. Define them here instead.
4631 'HAVE_SYS_UIO_H', 4624 'HAVE_SYS_UIO_H',
4632 ], 4625 ],
4633 'cflags': [ 4626 'cflags': [
4634 '--sysroot=<(android_ndk_sysroot)', 4627 '--sysroot=<(android_ndk_sysroot)',
4635 ], 4628 ],
(...skipping 273 matching lines...) Expand 10 before | Expand all | Expand 10 after
4909 '-fsanitize=address', 4902 '-fsanitize=address',
4910 '-mllvm -asan-globals=0', # http://crbug.com/352073 4903 '-mllvm -asan-globals=0', # http://crbug.com/352073
4911 '-gline-tables-only', 4904 '-gline-tables-only',
4912 ], 4905 ],
4913 }, 4906 },
4914 }], 4907 }],
4915 ['asan_coverage!=0', { 4908 ['asan_coverage!=0', {
4916 'target_conditions': [ 4909 'target_conditions': [
4917 ['_toolset=="target"', { 4910 ['_toolset=="target"', {
4918 'cflags': [ 4911 'cflags': [
4919 '-mllvm -asan-coverage=<(asan_coverage)', 4912 '-fsanitize-coverage=<(asan_coverage)',
4920 ], 4913 ],
4921 }], 4914 }],
4922 ], 4915 ],
4923 }], 4916 }],
4924 ], 4917 ],
4925 'target_conditions': [ 4918 'target_conditions': [
4926 ['_type!="static_library"', { 4919 ['_type!="static_library"', {
4927 'xcode_settings': {'OTHER_LDFLAGS': ['-Wl,-search_paths_first']}, 4920 'xcode_settings': {'OTHER_LDFLAGS': ['-Wl,-search_paths_first']},
4928 'conditions': [ 4921 'conditions': [
4929 ['asan==1', { 4922 ['asan==1', {
(...skipping 915 matching lines...) Expand 10 before | Expand all | Expand 10 after
5845 # settings in target dicts. SYMROOT is a special case, because many other 5838 # settings in target dicts. SYMROOT is a special case, because many other
5846 # Xcode variables depend on it, including variables such as 5839 # Xcode variables depend on it, including variables such as
5847 # PROJECT_DERIVED_FILE_DIR. When a source group corresponding to something 5840 # PROJECT_DERIVED_FILE_DIR. When a source group corresponding to something
5848 # like PROJECT_DERIVED_FILE_DIR is added to a project, in order for the 5841 # like PROJECT_DERIVED_FILE_DIR is added to a project, in order for the
5849 # files to appear (when present) in the UI as actual files and not red 5842 # files to appear (when present) in the UI as actual files and not red
5850 # red "missing file" proxies, the correct path to PROJECT_DERIVED_FILE_DIR, 5843 # red "missing file" proxies, the correct path to PROJECT_DERIVED_FILE_DIR,
5851 # and therefore SYMROOT, needs to be set at the project level. 5844 # and therefore SYMROOT, needs to be set at the project level.
5852 'SYMROOT': '<(DEPTH)/xcodebuild', 5845 'SYMROOT': '<(DEPTH)/xcodebuild',
5853 }, 5846 },
5854 } 5847 }
OLDNEW
« no previous file with comments | « no previous file | build/config/compiler/BUILD.gn » ('j') | chrome/installer/linux/rpm/expected_deps_i386 » ('J')

Powered by Google App Engine
This is Rietveld 408576698