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

Side by Side Diff: gyp/common_conditions.gypi

Issue 1510843003: Add RTTI to all sanitizers. (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: remove tab Created 5 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
« no previous file with comments | « no previous file | no next file » | 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 420 matching lines...) Expand 10 before | Expand all | Expand 10 after
431 'SKIA_DLL', 431 'SKIA_DLL',
432 'SKIA_IMPLEMENTATION=1', 432 'SKIA_IMPLEMENTATION=1',
433 ], 433 ],
434 }], 434 }],
435 # Enable asan, tsan, etc. 435 # Enable asan, tsan, etc.
436 [ 'skia_sanitizer', { 436 [ 'skia_sanitizer', {
437 'cflags': [ 437 'cflags': [
438 '-fsanitize=<(skia_sanitizer)', 438 '-fsanitize=<(skia_sanitizer)',
439 '-fno-sanitize-recover=<(skia_sanitizer)', 439 '-fno-sanitize-recover=<(skia_sanitizer)',
440 ], 440 ],
441 'cflags_cc!': [
442 '-fno-rtti'
443 ],
441 'ldflags': [ 444 'ldflags': [
442 '-fsanitize=<(skia_sanitizer)', 445 '-fsanitize=<(skia_sanitizer)',
443 ], 446 ],
444 'conditions' : [ 447 'conditions' : [
445 [ 'skia_sanitizer == "thread"', { 448 [ 'skia_sanitizer == "thread"', {
446 'defines': [ 'THREAD_SANITIZER' ], 449 'defines': [ 'THREAD_SANITIZER' ],
447 }], 450 }],
448 [ 'skia_sanitizer == "undefined"', {
449 'cflags_cc!': ['-fno-rtti'],
450 }],
451 ], 451 ],
452 }], 452 }],
453 [ 'skia_clang_build', { 453 [ 'skia_clang_build', {
454 'cflags_cc': [ 454 'cflags_cc': [
455 '-Wno-unknown-warning-option', # Allows unknown warnings. 455 '-Wno-unknown-warning-option', # Allows unknown warnings.
456 '-Wno-deprecated', # From Qt, via debugger (older C lang). 456 '-Wno-deprecated', # From Qt, via debugger (older C lang).
457 '-Wno-deprecated-register', # From Qt, via debugger (newer C lang). 457 '-Wno-deprecated-register', # From Qt, via debugger (newer C lang).
458 ], 458 ],
459 'cflags': [ 459 'cflags': [
460 # Extra warnings we like but that only Clang knows about. 460 # Extra warnings we like but that only Clang knows about.
(...skipping 186 matching lines...) Expand 10 before | Expand all | Expand 10 after
647 'LD_RUNPATH_SEARCH_PATHS': ['@executable_path/.'], 647 'LD_RUNPATH_SEARCH_PATHS': ['@executable_path/.'],
648 }, 648 },
649 }], 649 }],
650 650
651 ], # end 'conditions' 651 ], # end 'conditions'
652 # The Xcode SYMROOT must be at the root. See build/common.gypi in chromium for more details 652 # The Xcode SYMROOT must be at the root. See build/common.gypi in chromium for more details
653 'xcode_settings': { 653 'xcode_settings': {
654 'SYMROOT': '<(DEPTH)/xcodebuild', 654 'SYMROOT': '<(DEPTH)/xcodebuild',
655 }, 655 },
656 } 656 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698