Commit e196d00d authored by machenbach's avatar machenbach Committed by Commit bot

[test] Add back lsan leak detection

BUG=chromium:662388
TBR=yangguo@chromium.org, glider@chromium.org, titzer@chromium.org

Review-Url: https://codereview.chromium.org/2592663004
Cr-Commit-Position: refs/heads/master@{#42815}
parent 28dc846f
...@@ -61,6 +61,11 @@ template("v8_isolate_run") { ...@@ -61,6 +61,11 @@ template("v8_isolate_run") {
} else { } else {
asan = "0" asan = "0"
} }
if (is_lsan) {
lsan = "1"
} else {
lsan = "0"
}
if (is_msan) { if (is_msan) {
msan = "1" msan = "1"
} else { } else {
...@@ -158,6 +163,8 @@ template("v8_isolate_run") { ...@@ -158,6 +163,8 @@ template("v8_isolate_run") {
"--config-variable", "--config-variable",
"is_gn=1", "is_gn=1",
"--config-variable", "--config-variable",
"lsan=$lsan",
"--config-variable",
"msan=$msan", "msan=$msan",
"--config-variable", "--config-variable",
"tsan=$tsan", "tsan=$tsan",
......
...@@ -75,6 +75,7 @@ ...@@ -75,6 +75,7 @@
'--config-variable', 'has_valgrind=<(has_valgrind)', '--config-variable', 'has_valgrind=<(has_valgrind)',
'--config-variable', 'icu_use_data_file_flag=<(icu_use_data_file_flag)', '--config-variable', 'icu_use_data_file_flag=<(icu_use_data_file_flag)',
'--config-variable', 'is_gn=0', '--config-variable', 'is_gn=0',
'--config-variable', 'lsan=<(lsan)',
'--config-variable', 'msan=<(msan)', '--config-variable', 'msan=<(msan)',
'--config-variable', 'tsan=<(tsan)', '--config-variable', 'tsan=<(tsan)',
'--config-variable', 'coverage=<(coverage)', '--config-variable', 'coverage=<(coverage)',
......
# Do not add new suppressions below.
# TODO(machenbach): Delete this file as soon as it is empty.
# cctest
leak:v8::internal::Debug::NextAsyncTaskId
leak:v8::internal::wasm::DecodeWasmModule
leak:v8::internal::wasm::WasmInterpreter::WasmInterpreter
leak:v8::internal::WasmDebugInfo::SetBreakpoint
# debugger
leak:v8_inspector::WasmTranslation::TranslatorImpl::DisassemblingTranslator::AddFakeScript
leak:v8::internal::compiler::JumpThreading::ApplyForwarding
# mjsunit
leak:v8::internal::FuncNameInferrer::FuncNameInferrer
leak:v8::internal::JSArrayBuffer::SetupAllocatingData
# unittests
leak:v8::internal::Isolate::FindOrAllocatePerThreadDataForThisThread
...@@ -403,7 +403,14 @@ def SetupEnvironment(options): ...@@ -403,7 +403,14 @@ def SetupEnvironment(options):
) )
if options.asan: if options.asan:
os.environ['ASAN_OPTIONS'] = symbolizer os.environ['ASAN_OPTIONS'] = ":".join([
'detect_leaks=1',
symbolizer,
])
os.environ['LSAN_OPTIONS'] = ":".join([
'suppressions=%s' % os.path.join(
BASE_DIR, 'tools', 'memory', 'lsan', 'suppressions.txt'),
])
if options.sancov_dir: if options.sancov_dir:
assert os.path.exists(options.sancov_dir) assert os.path.exists(options.sancov_dir)
......
...@@ -27,5 +27,12 @@ ...@@ -27,5 +27,12 @@
], ],
}, },
}], }],
['lsan==1', {
'variables': {
'files': [
'../memory/lsan/suppressions.txt',
],
},
}],
], ],
} }
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment