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

Side by Side Diff: build/common.gypi

Issue 1665363002: Clang plugin to check that unstable types are not used in IPC. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Blacklist types instead 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/clang/BUILD.gn » ('j') | ipc/ipc_message_utils.h » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 2210 matching lines...) Expand 10 before | Expand all | Expand 10 after
2221 'clang_lib_path%': '<!(cd <(DEPTH) && pwd -P)/third_party/ll vm-build/Release+Asserts/lib/libFindBadConstructs.so', 2221 'clang_lib_path%': '<!(cd <(DEPTH) && pwd -P)/third_party/ll vm-build/Release+Asserts/lib/libFindBadConstructs.so',
2222 }], 2222 }],
2223 ], 2223 ],
2224 }, 2224 },
2225 'clang_dynlib_flags%': '-Xclang -load -Xclang <(clang_lib_path) ', 2225 'clang_dynlib_flags%': '-Xclang -load -Xclang <(clang_lib_path) ',
2226 }, { # OS == "win" 2226 }, { # OS == "win"
2227 # On Windows, the plugin is built directly into clang, so there's 2227 # On Windows, the plugin is built directly into clang, so there's
2228 # no need to load it dynamically. 2228 # no need to load it dynamically.
2229 'clang_dynlib_flags%': '', 2229 'clang_dynlib_flags%': '',
2230 }], 2230 }],
2231 ['OS=="android"', {
2232 'clang_plugin_args_ipc%': '-Xclang -plugin-arg-find-bad-constructs -Xclang check-ipc',
2233 }, {
2234 'clang_plugin_args_ipc%': '',
2235 }],
2231 ], 2236 ],
2232 'clang_plugin_args%': '-Xclang -plugin-arg-find-bad-constructs -Xclang check-templates ' 2237 'clang_plugin_args%': '-Xclang -plugin-arg-find-bad-constructs -Xclang check-templates '
2233 '-Xclang -plugin-arg-find-bad-constructs -Xclang follow-macro-expansio n ', 2238 '-Xclang -plugin-arg-find-bad-constructs -Xclang follow-macro-expansio n ',
2234 }, 2239 },
2235 # If you change these, also change build/config/clang/BUILD.gn. 2240 # If you change these, also change build/config/clang/BUILD.gn.
2236 'clang_chrome_plugins_flags%': 2241 'clang_chrome_plugins_flags%':
2237 '<(clang_dynlib_flags)' 2242 '<(clang_dynlib_flags)'
2238 '-Xclang -add-plugin -Xclang find-bad-constructs <(clang_plugin_args)' , 2243 '-Xclang -add-plugin -Xclang find-bad-constructs <(clang_plugin_args) <(clang_plugin_args_ipc)',
2239 }], 2244 }],
2240 ['asan==1 or msan==1 or lsan==1 or tsan==1', { 2245 ['asan==1 or msan==1 or lsan==1 or tsan==1', {
2241 'clang%': 1, 2246 'clang%': 1,
2242 'use_allocator%': 'none', 2247 'use_allocator%': 'none',
2243 'use_sanitizer_options%': 1, 2248 'use_sanitizer_options%': 1,
2244 }], 2249 }],
2245 2250
2246 ['OS=="linux" and asan==0 and msan==0 and lsan==0 and tsan==0', { 2251 ['OS=="linux" and asan==0 and msan==0 and lsan==0 and tsan==0', {
2247 # PNaCl toolchain Non-SFI build only supports linux OS build. 2252 # PNaCl toolchain Non-SFI build only supports linux OS build.
2248 # Also, it does not support sanitizers. 2253 # Also, it does not support sanitizers.
(...skipping 4135 matching lines...) Expand 10 before | Expand all | Expand 10 after
6384 # settings in target dicts. SYMROOT is a special case, because many other 6389 # settings in target dicts. SYMROOT is a special case, because many other
6385 # Xcode variables depend on it, including variables such as 6390 # Xcode variables depend on it, including variables such as
6386 # PROJECT_DERIVED_FILE_DIR. When a source group corresponding to something 6391 # PROJECT_DERIVED_FILE_DIR. When a source group corresponding to something
6387 # like PROJECT_DERIVED_FILE_DIR is added to a project, in order for the 6392 # like PROJECT_DERIVED_FILE_DIR is added to a project, in order for the
6388 # files to appear (when present) in the UI as actual files and not red 6393 # files to appear (when present) in the UI as actual files and not red
6389 # red "missing file" proxies, the correct path to PROJECT_DERIVED_FILE_DIR, 6394 # red "missing file" proxies, the correct path to PROJECT_DERIVED_FILE_DIR,
6390 # and therefore SYMROOT, needs to be set at the project level. 6395 # and therefore SYMROOT, needs to be set at the project level.
6391 'SYMROOT': '<(DEPTH)/xcodebuild', 6396 'SYMROOT': '<(DEPTH)/xcodebuild',
6392 }, 6397 },
6393 } 6398 }
OLDNEW
« no previous file with comments | « no previous file | build/config/clang/BUILD.gn » ('j') | ipc/ipc_message_utils.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698