<br><br><div class="gmail_quote">On Thu, Dec 15, 2011 at 4:21 PM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@google.com">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 class="gmail_quote"><div class="im">On Thu, Dec 15, 2011 at 4:14 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">

<br><br><div class="gmail_quote"><div>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><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></blockquote><div><br></div><div>yea, agree. </div><div>asan currently (almost) lacks this kind of testing and we will need to add more tests (not just this one). </div><div><br></div><div>For example, the current asan test suite has a test that invokes objdump and counts how many specific instructions are present in a given function.</div>
<div>FileCheck-like test should be much simpler. </div><div><br></div><div>--kcc </div><div><br></div><div><br></div><div><br></div><div> </div></div><br>