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

Unified Diff: build/common.gypi

Issue 871973004: win: Make asan work with isolates. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 11 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « base/base.isolate ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/common.gypi
diff --git a/build/common.gypi b/build/common.gypi
index dd0f6e9cfc14380005ef8954eb2905255cf6a275..f4d61339bd8aec29228b0f0ebfe75dfcc4852526 100644
--- a/build/common.gypi
+++ b/build/common.gypi
@@ -5549,52 +5549,6 @@
],
},
}],
- ['asan==1', {
- # ASan on Windows is a work in progress and very experimental.
- # See crbug.com/345874.
- 'VCCLCompilerTool': {
- 'AdditionalOptions': [
- '-fsanitize=address',
- ],
- 'AdditionalIncludeDirectories': [
- # MSVC needs to be able to find the sanitizer headers when
- # invoked via /fallback. This is critical for using macros
- # like ASAN_UNPOISON_MEMORY_REGION in files where we fall
- # back.
- '<(DEPTH)/<(make_clang_dir)/lib/clang/3.7.0/include_sanitizer',
- ],
- },
- 'VCLinkerTool': {
- 'AdditionalLibraryDirectories': [
- # TODO(hans): If make_clang_dir is absolute, this breaks.
- '<(DEPTH)/<(make_clang_dir)/lib/clang/3.7.0/lib/windows',
- ],
- },
- 'target_conditions': [
- ['component=="shared_library"', {
- 'VCLinkerTool': {
- 'AdditionalDependencies': [
- 'clang_rt.asan_dynamic-i386.lib',
- 'clang_rt.asan_dynamic_runtime_thunk-i386.lib',
- ],
- },
- }],
- ['_type=="executable" and component=="static_library"', {
- 'VCLinkerTool': {
- 'AdditionalDependencies': [
- 'clang_rt.asan-i386.lib',
- ],
- },
- }],
- ['(_type=="shared_library" or _type=="loadable_module") and component=="static_library"', {
- 'VCLinkerTool': {
- 'AdditionalDependencies': [
- 'clang_rt.asan_dll_thunk-i386.lib',
- ],
- },
- }],
- ],
- }],
],
},
},
@@ -5629,12 +5583,61 @@
'/ignore:4221',
'/nxcompat',
],
- 'conditions': [
- ['syzyasan==0', {
- 'AdditionalOptions': ['/largeaddressaware'],
- }],
- ],
},
+ 'conditions': [
+ ['syzyasan==0', {
+ 'VCLinkerTool': {
+ 'AdditionalOptions': ['/largeaddressaware'],
+ },
+ }],
+ ['asan==1', {
+ # TODO(asan/win): Move this down into the general
+ # win-target_defaults section once the 64-bit asan runtime
+ # exists. See crbug.com/345874.
+ 'VCCLCompilerTool': {
+ 'AdditionalOptions': [
+ '-fsanitize=address',
+ ],
+ 'AdditionalIncludeDirectories': [
+ # MSVC needs to be able to find the sanitizer headers when
+ # invoked via /fallback. This is critical for using macros
+ # like ASAN_UNPOISON_MEMORY_REGION in files where we fall
+ # back.
+ '<(DEPTH)/<(make_clang_dir)/lib/clang/3.7.0/include_sanitizer',
+ ],
+ },
+ 'VCLinkerTool': {
+ 'AdditionalLibraryDirectories': [
+ # TODO(hans): If make_clang_dir is absolute, this breaks.
+ '<(DEPTH)/<(make_clang_dir)/lib/clang/3.7.0/lib/windows',
+ ],
+ },
+ 'target_conditions': [
+ ['component=="shared_library"', {
+ 'VCLinkerTool': {
+ 'AdditionalDependencies': [
+ 'clang_rt.asan_dynamic-i386.lib',
+ 'clang_rt.asan_dynamic_runtime_thunk-i386.lib',
+ ],
+ },
+ }],
+ ['_type=="executable" and component=="static_library"', {
+ 'VCLinkerTool': {
+ 'AdditionalDependencies': [
+ 'clang_rt.asan-i386.lib',
+ ],
+ },
+ }],
+ ['(_type=="shared_library" or _type=="loadable_module") and component=="static_library"', {
+ 'VCLinkerTool': {
+ 'AdditionalDependencies': [
+ 'clang_rt.asan_dll_thunk-i386.lib',
+ ],
+ },
+ }],
+ ],
+ }],
+ ],
},
},
'x64_Base': {
« no previous file with comments | « base/base.isolate ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698