<div dir="ltr">I was mostly going for consistency, since other tests also overwrite the files. I can revert that name change.<div><br></div><div>Want me to change the other tests too, so that they generate two files instead of one + overwrite?</div><div><br></div><div>  Filipe</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Dec 16, 2014 at 7:31 PM, Alexey Samsonov <span dir="ltr"><<a href="mailto:vonosmas@gmail.com" target="_blank">vonosmas@gmail.com</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">================<br>
Comment at: test/lsan/TestCases/suppressions_file.cc:7<br>
@@ -6,3 +6,3 @@<br>
<br>
-// RUN: echo "leak:%t" > %t.supp2<br>
-// RUN: LSAN_OPTIONS=$LSAN_BASE:suppressions="%t.supp2":symbolize=false %run %t<br>
+// RUN: echo "leak:%t" > %t.supp<br>
+// RUN: LSAN_OPTIONS=$LSAN_BASE:suppressions='%t.supp':symbolize=false %run %t<br>
----------------<br>
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.<br>
<br>
<a href="http://reviews.llvm.org/D6680" target="_blank">http://reviews.llvm.org/D6680</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
<br>
<br>
</blockquote></div></div></div>