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

Unified Diff: scripts/slave/runtest.py

Issue 941433004: Remove LSan options/suppressions from recipe configs. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: rebase Created 5 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 | « scripts/slave/recipes/webrtc/standalone.expected/tryserver_webrtc_linux_asan.json ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: scripts/slave/runtest.py
diff --git a/scripts/slave/runtest.py b/scripts/slave/runtest.py
index 6c093435d00e49f4d3784cf2d774f7ce6387b276..8ee880eeecf27013b5fc8a7b9779a28e892ab489 100755
--- a/scripts/slave/runtest.py
+++ b/scripts/slave/runtest.py
@@ -1734,9 +1734,7 @@ def _ConfigureSanitizerTools(options, args, extra_env):
# LeakSanitizer
if options.enable_lsan:
# Symbolization options set here take effect only for standalone LSan.
- lsan_options = symbolization_options + \
- ['suppressions=%s' % options.lsan_suppressions_file,
- 'print_suppressions=1']
+ lsan_options = symbolization_options
AddToExistingEnv(extra_env, 'LSAN_OPTIONS', lsan_options)
# Disable sandboxing under LSan.
@@ -1919,10 +1917,6 @@ def main():
help='Enable memory leak detection (LeakSanitizer). '
'Can also be enabled with the factory '
'property "lsan" (deprecated).')
- option_parser.add_option('--lsan-suppressions-file',
- default='src/tools/lsan/suppressions.txt',
- help='Suppression file for LeakSanitizer. '
- 'Default: %default.')
option_parser.add_option('--enable-msan', action='store_true', default=False,
help='Enable uninitialized memory reads detection '
'(MemorySanitizer). Can also enabled with the '
« no previous file with comments | « scripts/slave/recipes/webrtc/standalone.expected/tryserver_webrtc_linux_asan.json ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698