<div style="font-family: arial, helvetica, sans-serif; font-size: 10pt"><br><br><div class="gmail_quote">On Fri, Dec 7, 2012 at 10:26 PM, Dmitry Vyukov <span dir="ltr"><<a href="mailto:dvyukov@google.com" target="_blank">dvyukov@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="HOEnZb"><div class="h5">On Fri, Dec 7, 2012 at 10:23 PM, Dmitry Vyukov <<a href="mailto:dvyukov@google.com">dvyukov@google.com</a>> wrote:<br>

> On Fri, Dec 7, 2012 at 10:12 PM, Kostya Serebryany <<a href="mailto:kcc@google.com">kcc@google.com</a>> wrote:<br>
>><br>
>><br>
>> On Fri, Dec 7, 2012 at 8:23 PM, Dmitry Vyukov <<a href="mailto:dvyukov@google.com">dvyukov@google.com</a>> wrote:<br>
>>><br>
>>> Please check lint before committing and remove debug output from<br>
>>> SanitizerCommon.SanitizerSetThreadName test.<br>
>><br>
>> done, sorry. r169620.<br>
>><br>
>>><br>
>>> Also, can we do something with allocator tests? Tsan tests were<br>
>>> running for 1.5sec, not they run for 30sec and 29 are occupied by<br>
>>> allocator tests. Are we are testing in 4 configurations.<br>
>><br>
>><br>
>> I am not seeing this:<br>
>> %<br>
>> ./projects/compiler-rt/lib/sanitizer_common/tests/Release/SanitizerUnitTest<br>
>> [==========] 39 tests from 3 test cases ran. (4924 ms total)<br>
>><br>
>> If we reduce the load in the tests we will not be catching corner cases that<br>
>> only appear under a load.<br>
>><br>
>> When you run 'make -j32 check-sanitizer', gtest is called in sharded mode,<br>
>> so effectively the running time is less than 1 second.<br>
>><br>
>> I guess, we are also running the allocator tests inside the tsan tests (do<br>
>> we?), which would be wrong anyway.<br>
><br>
><br>
> Yes, we running all sanitizer unit tests in tsan.<br>
<br>
</div></div>We testing both release and debug mode. I guess you run only release.<br>
In tsan we test clang debug + clang release + gcc debug + gcc release.<br></blockquote><div><br></div><div>I see. This is the part of workflow unsupported by the cmake, so no one except you (and sometimes me) sees it. </div>
<div>Do you really need to run the allocator (and other sanitizer_common) tests in those 4 modes? </div><div> </div><div>--kcc </div></div><br></div>