Chromium Code Reviews| 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'] |