[PATCH] D38026: Make LSan compliant with recovery mode when running on top of ASan

Kostya Serebryany via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 19 09:05:14 PDT 2017


kcc added inline comments.


================
Comment at: test/asan/TestCases/recoverable-lsan.cc:4
+// RUN: %env_asan_opts="detect_leaks=1:halt_on_error=0" %run %t 2>&1 | FileCheck
+// %s RUN: %env_asan_opts="detect_leaks=1:halt_on_error=1" not %run %t 2>&1 |
+// FileCheck %s
----------------
Please format. 
Why to you need to set detect_leaks (it's the default)?
Also, please add a line w/o halt_on_error flag to test the default behavior. 


https://reviews.llvm.org/D38026





More information about the llvm-commits mailing list