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

Unified Diff: build/standalone.gypi

Issue 1727383002: Revert of [coverage] Filter some files from instrumentation. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « build/coverage_wrapper.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/standalone.gypi
diff --git a/build/standalone.gypi b/build/standalone.gypi
index 80efd775989cb6d5e585dbbe409a2a777832836b..5269b95f64dbd9cf2f6ecba7c9c0d8764a4bf550 100644
--- a/build/standalone.gypi
+++ b/build/standalone.gypi
@@ -48,12 +48,6 @@
'release_extra_cflags%': '',
'variables': {
'variables': {
- # goma settings.
- # 1 to use goma.
- # If no gomadir is set, it uses the default gomadir.
- 'use_goma%': 0,
- 'gomadir%': '',
-
'variables': {
'conditions': [
['OS=="linux" or OS=="freebsd" or OS=="openbsd" or \
@@ -76,23 +70,12 @@
# Instrument for code coverage with gcov.
'coverage%': 0,
-
- 'conditions': [
- # Set default gomadir.
- ['OS=="win"', {
- 'gomadir': 'c:\\goma\\goma-win',
- }, {
- 'gomadir': '<!(/bin/echo -n ${HOME}/goma)',
- }],
- ],
},
'base_dir%': '<(base_dir)',
'host_arch%': '<(host_arch)',
'target_arch%': '<(target_arch)',
'v8_target_arch%': '<(target_arch)',
'coverage%': '<(coverage)',
- 'use_goma%': '<(use_goma)',
- 'gomadir%': '<(gomadir)',
'asan%': 0,
'lsan%': 0,
'msan%': 0,
@@ -121,10 +104,22 @@
# TODO(machenbach): Only configured for windows.
'fastbuild%': 0,
+ # goma settings.
+ # 1 to use goma.
+ # If no gomadir is set, it uses the default gomadir.
+ 'use_goma%': 0,
+ 'gomadir%': '',
+
# Check if valgrind directories are present.
'has_valgrind%': '<!pymod_do_main(has_valgrind)',
'conditions': [
+ # Set default gomadir.
+ ['OS=="win"', {
+ 'gomadir': 'c:\\goma\\goma-win',
+ }, {
+ 'gomadir': '<!(/bin/echo -n ${HOME}/goma)',
+ }],
['host_arch!="ppc" and host_arch!="ppc64" and host_arch!="ppc64le" and host_arch!="s390" and host_arch!="s390x" and \
coverage==0', {
'host_clang%': 1,
@@ -367,23 +362,6 @@
'host_cc': '<!(which gcc)',
'host_cxx': '<!(which g++)',
}],
- ['use_goma==1 and ("<(GENERATOR)"=="ninja" or clang==1)', {
- 'conditions': [
- ['coverage==1', {
- 'cc_wrapper': '<(base_dir)/build/coverage_wrapper.py <(gomadir)/gomacc',
- }, {
- 'cc_wrapper': '<(gomadir)/gomacc',
- }],
- ],
- }, {
- 'conditions': [
- ['coverage==1', {
- 'cc_wrapper': '<(base_dir)/build/coverage_wrapper.py',
- }, {
- 'cc_wrapper': 0,
- }],
- ],
- }],
],
# Default ARM variable settings.
'arm_version%': 'default',
@@ -1283,12 +1261,12 @@
# TODO(yyanagisawa): supports GENERATOR==make
# make generator doesn't support CC_wrapper without CC
# in make_global_settings yet.
- ['cc_wrapper!=0', {
+ ['use_goma==1 and ("<(GENERATOR)"=="ninja" or clang==1)', {
'make_global_settings': [
- ['CC_wrapper', '<(cc_wrapper)'],
- ['CXX_wrapper', '<(cc_wrapper)'],
- ['CC.host_wrapper', '<(cc_wrapper)'],
- ['CXX.host_wrapper', '<(cc_wrapper)'],
+ ['CC_wrapper', '<(gomadir)/gomacc'],
+ ['CXX_wrapper', '<(gomadir)/gomacc'],
+ ['CC.host_wrapper', '<(gomadir)/gomacc'],
+ ['CXX.host_wrapper', '<(gomadir)/gomacc'],
],
}],
['use_lto==1', {
« no previous file with comments | « build/coverage_wrapper.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698