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

Side by Side Diff: build/standalone.gypi

Issue 802583003: [test] Make msan work for v8 stand-alone. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Rework from scratch Created 5 years, 8 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 | 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 2012 the V8 project authors. All rights reserved. 1 # Copyright 2012 the V8 project authors. All rights reserved.
2 # Redistribution and use in source and binary forms, with or without 2 # Redistribution and use in source and binary forms, with or without
3 # modification, are permitted provided that the following conditions are 3 # modification, are permitted provided that the following conditions are
4 # met: 4 # met:
5 # 5 #
6 # * Redistributions of source code must retain the above copyright 6 # * Redistributions of source code must retain the above copyright
7 # notice, this list of conditions and the following disclaimer. 7 # notice, this list of conditions and the following disclaimer.
8 # * Redistributions in binary form must reproduce the above 8 # * Redistributions in binary form must reproduce the above
9 # copyright notice, this list of conditions and the following 9 # copyright notice, this list of conditions and the following
10 # disclaimer in the documentation and/or other materials provided 10 # disclaimer in the documentation and/or other materials provided
(...skipping 17 matching lines...) Expand all
28 # Definitions to be used when building stand-alone V8 binaries. 28 # Definitions to be used when building stand-alone V8 binaries.
29 29
30 { 30 {
31 # We need to include toolchain.gypi here for third-party sources that don't 31 # We need to include toolchain.gypi here for third-party sources that don't
32 # directly include it themselves. 32 # directly include it themselves.
33 'includes': ['toolchain.gypi'], 33 'includes': ['toolchain.gypi'],
34 'variables': { 34 'variables': {
35 'component%': 'static_library', 35 'component%': 'static_library',
36 'clang_dir%': 'third_party/llvm-build/Release+Asserts', 36 'clang_dir%': 'third_party/llvm-build/Release+Asserts',
37 'clang_xcode%': 0, 37 'clang_xcode%': 0,
38 # Track where uninitialized memory originates from. From fastest to
39 # slowest: 0 - no tracking, 1 - track only the initial allocation site, 2
40 # - track the chain of stores leading from allocation site to use site.
41 'msan_track_origins%': 1,
earthdok 2015/05/05 14:40:47 I suggest '2' here. The performance difference is
38 'visibility%': 'hidden', 42 'visibility%': 'hidden',
39 'v8_enable_backtrace%': 0, 43 'v8_enable_backtrace%': 0,
40 'v8_enable_i18n_support%': 1, 44 'v8_enable_i18n_support%': 1,
41 'v8_deprecation_warnings': 1, 45 'v8_deprecation_warnings': 1,
42 'msvs_multi_core_compile%': '1', 46 'msvs_multi_core_compile%': '1',
43 'mac_deployment_target%': '10.5', 47 'mac_deployment_target%': '10.5',
44 'release_extra_cflags%': '', 48 'release_extra_cflags%': '',
45 'variables': { 49 'variables': {
46 'variables': { 50 'variables': {
47 'variables': { 51 'variables': {
(...skipping 260 matching lines...) Expand 10 before | Expand all | Expand 10 after
308 ], 312 ],
309 'ldflags': [ 313 'ldflags': [
310 '-fsanitize=thread', 314 '-fsanitize=thread',
311 '-pie', 315 '-pie',
312 ], 316 ],
313 'defines': [ 317 'defines': [
314 'THREAD_SANITIZER', 318 'THREAD_SANITIZER',
315 ], 319 ],
316 }, 320 },
317 }], 321 }],
322 ['msan==1 and OS!="mac"', {
323 'target_defaults': {
324 'cflags_cc+': [
325 '-fno-omit-frame-pointer',
326 '-gline-tables-only',
327 '-fsanitize=memory',
328 '-fsanitize-memory-track-origins=<(msan_track_origins)',
329 '-fPIC',
330 ],
331 'cflags+': [
Michael Achenbach 2015/04/27 09:08:39 Without the additional cflags -fPIC and the remova
earthdok 2015/05/05 14:40:47 This is very strange. -fPIC should be implied by -
332 '-fPIC',
333 ],
334 'cflags!': [
335 '-fno-exceptions',
336 '-fomit-frame-pointer',
337 ],
338 'ldflags': [
339 '-fsanitize=memory',
340 ],
341 'defines': [
342 'MEMORY_SANITIZER',
343 ],
344 'dependencies': [
345 # Use libc++ (third_party/libc++ and third_party/libc++abi) instead of
346 # stdlibc++ as standard library. This is intended to use for instrumen ted
347 # builds.
348 '<(DEPTH)/buildtools/third_party/libc++/libc++.gyp:libcxx_proxy',
349 ],
350 },
351 }],
318 ['asan==1 and OS=="mac"', { 352 ['asan==1 and OS=="mac"', {
319 'target_defaults': { 353 'target_defaults': {
320 'xcode_settings': { 354 'xcode_settings': {
321 'OTHER_CFLAGS+': [ 355 'OTHER_CFLAGS+': [
322 '-fno-omit-frame-pointer', 356 '-fno-omit-frame-pointer',
323 '-gline-tables-only', 357 '-gline-tables-only',
324 '-fsanitize=address', 358 '-fsanitize=address',
325 '-w', # http://crbug.com/162783 359 '-w', # http://crbug.com/162783
326 ], 360 ],
327 'OTHER_CFLAGS!': [ 361 'OTHER_CFLAGS!': [
(...skipping 268 matching lines...) Expand 10 before | Expand all | Expand 10 after
596 ['use_goma==1 and ("<(GENERATOR)"=="ninja" or clang==1)', { 630 ['use_goma==1 and ("<(GENERATOR)"=="ninja" or clang==1)', {
597 'make_global_settings': [ 631 'make_global_settings': [
598 ['CC_wrapper', '<(gomadir)/gomacc'], 632 ['CC_wrapper', '<(gomadir)/gomacc'],
599 ['CXX_wrapper', '<(gomadir)/gomacc'], 633 ['CXX_wrapper', '<(gomadir)/gomacc'],
600 ['CC.host_wrapper', '<(gomadir)/gomacc'], 634 ['CC.host_wrapper', '<(gomadir)/gomacc'],
601 ['CXX.host_wrapper', '<(gomadir)/gomacc'], 635 ['CXX.host_wrapper', '<(gomadir)/gomacc'],
602 ], 636 ],
603 }], 637 }],
604 ], 638 ],
605 } 639 }
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