[PATCH] Fix suppressions file usage

Alexey Samsonov vonosmas at gmail.com
Tue Dec 16 11:31:41 PST 2014


================
Comment at: test/lsan/TestCases/suppressions_file.cc:7
@@ -6,3 +6,3 @@
 
-// RUN: echo "leak:%t" > %t.supp2
-// RUN: LSAN_OPTIONS=$LSAN_BASE:suppressions="%t.supp2":symbolize=false %run %t
+// RUN: echo "leak:%t" > %t.supp
+// RUN: LSAN_OPTIONS=$LSAN_BASE:suppressions='%t.supp':symbolize=false %run %t
----------------
Why would you need to change extension to .supp here? I think it's fine to have two different files in this test instead of overwriting a single file.

http://reviews.llvm.org/D6680

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list