<div class="gmail_quote">On Thu, Dec 15, 2011 at 4:14 PM, Kostya Serebryany <span dir="ltr"><<a href="mailto:kcc@google.com">kcc@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br><br><div class="gmail_quote"><div class="im">On Thu, Dec 15, 2011 at 4:11 PM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@google.com" target="_blank">chandlerc@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div><div class="gmail_quote">On Thu, Dec 15, 2011 at 1:59 PM, Kostya Serebryany <span dir="ltr"><<a href="mailto:kcc@google.com" target="_blank">kcc@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<div>[asan] fix a bug (issue 19) where dlclose and the following mmap caused a false positive. compiler part.<br></div></blockquote></div><br></div><div>No test case? </div></blockquote><div><br></div></div><div>There is a full (compile-link-run) test: projects/compiler-rt/lib/asan/tests/dlclose-test*</div>

<div>I do not see any need for a separate compiler-only test, do you?</div></div></blockquote><div><br></div><div>Not everyone will be able to run those tests, and certainly not on every platform. The nice thing about regression testing the actual compiler transformations in the LLVM regtest suite is that they are completely architecture independent, and isolate the functionality under test. Other developers need to be able to run the tests and ensure their changes don't break your functionality.</div>
<div><br></div><div>It is expected that nearly all of the logic of a compiler transformation at the IR level is tested using FileCheck and the LLVM regtest suite. Only final integration testing and testing for strange system interactions should be pushed down to external projects' test suites.</div>
</div>