<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On 30 January 2014 12:49, Alexey Samsonov <span dir="ltr"><<a href="mailto:samsonov@google.com" target="_blank">samsonov@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Did you actually upload the new version? I don't see it in Phab.<br>
</blockquote><div><br></div><div>No, I'm still playing with the options, and the main change is in Clang, so not relevant to this RT phab review. Attached is the Clang patch.</div><div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
  The message about detect_leaks is irrelevant here, it's just a warning. However, we should probably turn on leak checking in ASan unit tests only on supported platforms to reduce noise. See compiler-rt/lib/asan/lit_tests/Unit/<a href="http://lit.site.cfg.in" target="_blank">lit.site.cfg.in</a>. However, you may not bother about it at the moment.<br>
</blockquote><div><br></div><div>Right, I'll try and disable just the leak check, something like:</div><div><br></div><div><div>if config.host_os == 'Linux' and config.arch == 'x86_64':</div><div>  config.environment['ASAN_OPTIONS'] = 'detect_leaks=1'</div>
</div><div><br></div><div>Right now, I'm building the whole thing, will take a while. </div><div><br></div><div>I found it hard to build just the unit tests. Even if I delete the test binary, or the objects, and "make check-asan", it doesn't rebuild the unit test (but it does run it). Maybe something wrong with the cmake files?</div>
<div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">  Can you invoke the test cases directly (run the unit test binary with specifying --gtest_filter=AddressSanitizer.ThreadedMallocStressTest) and see what the problem is? These tests have substantial memory requirements, so maybe there's just not enough RAM on your machine. I think we have ASAN_LOW_MEMORY in ASan runtime compile definition for that.<br>
</blockquote><div></div></div><br></div><div class="gmail_extra">I noticed that they segfault right after printing the message, so the message is probably a red herring. I will run them in debug mode.</div><div class="gmail_extra">
<br></div><div class="gmail_extra">cheers,</div><div class="gmail_extra">--renato</div><div class="gmail_extra"><br></div></div>