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

Unified Diff: scripts/slave/recipe_modules/chromium/config.py

Issue 612753004: Create MSan config for Chromium recipe module + add WebRTC MSan recipe. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: Created 6 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 side-by-side diff with in-line comments
Download patch
Index: scripts/slave/recipe_modules/chromium/config.py
diff --git a/scripts/slave/recipe_modules/chromium/config.py b/scripts/slave/recipe_modules/chromium/config.py
index f452a461cd170e3f944673e0d1f5c8e78572a88a..507fdc4a022916ba6bda97ea9dbc164cdfc1a2d0 100644
--- a/scripts/slave/recipe_modules/chromium/config.py
+++ b/scripts/slave/recipe_modules/chromium/config.py
@@ -333,6 +333,14 @@ def asan(c):
def no_lsan(c):
c.gyp_env.GYP_DEFINES['lsan'] = 0
+@config_ctx()
+def msan(c):
+ gyp_defs = c.gyp_env.GYP_DEFINES
+ gyp_defs['msan'] = 1
+ gyp_defs['msan_track_origins'] = 0
earthdok 2014/09/29 13:25:08 msan_track_origins=0 should not be a part of the c
kjellander_chromium 2014/09/30 10:56:40 That makes sense, I left that variable in the exis
+ gyp_defs['use_instrumented_libraries'] = 1
earthdok 2014/09/29 13:25:08 I'm also against adding instrumented libraries to
kjellander_chromium 2014/09/30 10:56:40 Fair enough. I added it as as separate config and
+ gyp_defs['instrumented_libraries_jobs'] = 10
+
@config_ctx(group='memory_tool')
def memcheck(c):
_memory_tool(c, 'memcheck')
@@ -400,6 +408,10 @@ def chromium_asan(c):
c.runtests.test_args.append('--test-launcher-batch-limit=1')
c.runtests.test_args.append('--test-launcher-print-test-stdio=always')
+@config_ctx(includes=['ninja', 'clang', 'goma', 'msan'])
+def chromium_msan(c):
+ c.compile_py.default_targets = ['All', 'chromium_builder_tests']
+
@config_ctx(includes=['ninja', 'clang', 'goma', 'syzyasan'])
def chromium_syzyasan(c):
c.compile_py.default_targets = ['All', 'chromium_builder_tests']
« no previous file with comments | « scripts/slave/recipe_modules/chromium/chromium_memory_fyi.py ('k') | scripts/slave/recipe_modules/chromium_tests/api.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698