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

Unified Diff: tools/mb/mb_config.pyl

Issue 1648803002: Add valgrind config into tools/mb/mb_config.pyl (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/mb/mb_config.pyl
diff --git a/tools/mb/mb_config.pyl b/tools/mb/mb_config.pyl
index a70c0d6d4dc931ede3b6ec7100f2f88bbf42b55e..7edf19639465c75dd30e7952f62a7900625d19bf 100644
--- a/tools/mb/mb_config.pyl
+++ b/tools/mb/mb_config.pyl
@@ -74,6 +74,7 @@
'libfuzzer_upload_bot_msan': ['gn', 'release', 'libfuzzer', 'msan', 'proprietary_codecs'],
'drfuzz_upload_bot_win32': ['gn', 'release', 'drmemory', 'drfuzz', 'x86', 'proprietary_codecs'],
'drfuzz_upload_bot_win64': ['gn', 'release', 'drmemory', 'drfuzz', 'x64', 'proprietary_codecs'],
+ 'gyp_valgrind_bot': ['gyp', 'release', 'x64', 'valgrind'],
# This is just for completeness; any bot that uses this config
# should never actually run MB.
@@ -415,6 +416,11 @@
'drmemory': {
'gn_args': 'is_component_build=true enable_iterator_debugging=false',
'gyp_defines': 'component=shared_library build_for_tool=drmemory',
+ },
+
+ 'valgrind': {
+ # TODO: add gn_args for 'build_for_tool=memcheck'
+ 'gyp_defines': 'build_for_tool=memcheck',
}
},
@@ -551,6 +557,14 @@
'Android Builder': 'gyp_release_bot_android',
'WebKit Android (Nexus4)': 'gyp_release_bot_android',
},
+ 'chromium.memory.fyi': {
+ 'Chromium Linux Builder (valgrind)': 'gyp_valgrind_bot',
+ 'Linux Tests (valgrind)(1)': 'gyp_valgrind_bot',
+ 'Linux Tests (valgrind)(2)': 'gyp_valgrind_bot',
+ 'Linux Tests (valgrind)(3)': 'gyp_valgrind_bot',
+ 'Linux Tests (valgrind)(4)': 'gyp_valgrind_bot',
+ 'Linux Tests (valgrind)(5)': 'gyp_valgrind_bot',
+ },
'chromium.fyi': {
'CFI Linux': 'gn_cfi_release_bot',
'CFI Linux ToT': 'gn_cfi_release_bot',
« 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