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

Side by Side Diff: gyp/common_conditions.gypi

Issue 2293063002: Detect TSAN instead of relying on #ifdef THREAD_SANITIZER. (Closed)
Patch Set: Created 4 years, 3 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 | src/core/SkSharedMutex.cpp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2015 Google Inc. 1 # Copyright 2015 Google Inc.
2 # 2 #
3 # Use of this source code is governed by a BSD-style license that can be 3 # Use of this source code is governed by a BSD-style license that can be
4 # found in the LICENSE file. 4 # found in the LICENSE file.
5 5
6 # conditions used in both common.gypi and skia.gyp in chromium 6 # conditions used in both common.gypi and skia.gyp in chromium
7 # 7 #
8 { 8 {
9 'defines': [ 9 'defines': [
10 'SK_ALLOW_STATIC_GLOBAL_INITIALIZERS=<(skia_static_initializers)', 10 'SK_ALLOW_STATIC_GLOBAL_INITIALIZERS=<(skia_static_initializers)',
(...skipping 446 matching lines...) Expand 10 before | Expand all | Expand 10 after
457 [ 'skia_sanitizer', { 457 [ 'skia_sanitizer', {
458 'cflags_cc!': [ '-fno-rtti' ], # vptr needs r tti 458 'cflags_cc!': [ '-fno-rtti' ], # vptr needs r tti
459 'cflags': [ 459 'cflags': [
460 '-fsanitize=<(skia_sanitizer)', # Turn on sani tizers. 460 '-fsanitize=<(skia_sanitizer)', # Turn on sani tizers.
461 '-fno-sanitize-recover=<(skia_sanitizer)', # Make any fai lure fatal. 461 '-fno-sanitize-recover=<(skia_sanitizer)', # Make any fai lure fatal.
462 '-fsanitize-blacklist=<(skia_sanitizer_blacklist)', # Compile in o ur blacklist. 462 '-fsanitize-blacklist=<(skia_sanitizer_blacklist)', # Compile in o ur blacklist.
463 '-include <(skia_sanitizer_blacklist)', # Make every . cpp depend on it. 463 '-include <(skia_sanitizer_blacklist)', # Make every . cpp depend on it.
464 ], 464 ],
465 'ldflags': [ '-fsanitize=<(skia_sanitizer)' ], 465 'ldflags': [ '-fsanitize=<(skia_sanitizer)' ],
466 'conditions' : [ 466 'conditions' : [
467 [ 'skia_sanitizer == "thread"', {
468 'defines': [ 'THREAD_SANITIZER' ],
469 }],
470 [ 'skia_sanitizer == "memory"', { 467 [ 'skia_sanitizer == "memory"', {
471 'cflags': [ 468 'cflags': [
472 '-O1', 469 '-O1',
473 '-fsanitize-memory-track-origins', 470 '-fsanitize-memory-track-origins',
474 ], 471 ],
475 }], 472 }],
476 ], 473 ],
477 }], 474 }],
478 [ 'skia_clang_build', { 475 [ 'skia_clang_build', {
479 'cflags_cc': [ 476 'cflags_cc': [
(...skipping 221 matching lines...) Expand 10 before | Expand all | Expand 10 after
701 'LD_RUNPATH_SEARCH_PATHS': ['@executable_path/.'], 698 'LD_RUNPATH_SEARCH_PATHS': ['@executable_path/.'],
702 }, 699 },
703 }], 700 }],
704 701
705 ], # end 'conditions' 702 ], # end 'conditions'
706 # The Xcode SYMROOT must be at the root. See build/common.gypi in chromium for more details 703 # The Xcode SYMROOT must be at the root. See build/common.gypi in chromium for more details
707 'xcode_settings': { 704 'xcode_settings': {
708 'SYMROOT': '<(DEPTH)/xcodebuild', 705 'SYMROOT': '<(DEPTH)/xcodebuild',
709 }, 706 },
710 } 707 }
OLDNEW
« no previous file with comments | « no previous file | src/core/SkSharedMutex.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698