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

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: Refactored out instrumented_libraries config rebased 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 26a19d45e56811f6687fced2d0bc2edb1eb94e0e..5b26a0ab8635ca2f04e4890e8536f7238ed708e9 100644
--- a/scripts/slave/recipe_modules/chromium/config.py
+++ b/scripts/slave/recipe_modules/chromium/config.py
@@ -340,6 +340,17 @@ def asan(c):
def no_lsan(c):
c.gyp_env.GYP_DEFINES['lsan'] = 0
+@config_ctx(deps=['compiler'])
+def msan(c):
+ if 'clang' not in c.compile_py.compiler: # pragma: no cover
+ raise BadConf('msan requires clang')
+ c.gyp_env.GYP_DEFINES['msan'] = 1
+
+@config_ctx()
+def instrumented_libraries(c):
+ c.gyp_env.GYP_DEFINES['use_instrumented_libraries'] = 1
+ c.gyp_env.GYP_DEFINES['instrumented_libraries_jobs'] = 10
+
@config_ctx(group='memory_tool')
def memcheck(c):
_memory_tool(c, 'memcheck')
@@ -405,6 +416,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