<div dir="ltr">Hi Kuba,<br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jul 23, 2015 at 7:52 AM, Kuba Brecka <span dir="ltr"><<a href="mailto:kuba.brecka@gmail.com" target="_blank">kuba.brecka@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">kubabrecka updated this revision to Diff 30485.<br>
kubabrecka added a comment.<br>
<br>
Kostya, thanks for dealing with this!<br>
<br>
Changed `(SANITIZER_MAC == 1)` to just `SANITIZER_MAC`.  Removed the whole `__asan_default_options` thing for ASan unit tests and instead moved in into `test/asan/Unit/<a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__lit.site.cfg.in&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=mQ4LZ2PUj9hpadE3cDHZnIdEwhEBrbAstXeMaFoB9tg&m=lDEwkSm5eQ7qeyWKaaRj1YOyKVsXtFiGTeouEKlBF5E&s=QdGuZvVOmjb3jelNGUeuEyIAyqmlrNEhJs7Q3wB4JLU&e=" rel="noreferrer" target="_blank">lit.site.cfg.in</a>`.  Addressed other comments.<br>
<span class=""><br>
> I don't think the "See also ..." part is useful, and will quickly become annoying as other platforms are added.<br>
<br>
<br>
</span>I think it's important to note that the two tests are related (if you're editing one, you should be aware of the other).  Can we keep this?<br></blockquote><div>No problem on my end if you prefer to have it.</div><div>But maybe we don't need to repeat the OS name on the TestCases/*/abort_on_error-*.cc tests, no?</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">> There will be a tax: please work with filcab to remove config.environment['ASAN_OPTIONS'] = 'symbolize_vs_style=false from the main path in lit.cfg<br>
<br>
>  I want it to be done only for those platforms where he needs this to happen -- but not by default and not on Linux<br>
<br>
<br>
</span>Filipe, could you tell me on what platforms do you need to override `symbolize_vs_style`?  Also, it looks to me that it's always `false` by default for all platforms...  Are you talking about out-of-tree modifications of the defaults?<br></blockquote><div>It's false on all platforms in open source. I don't know if anyone is shipping the sanitizers on Windows yet, so UX should probably be centered around whatever Google does with it. If we start shipping it with llvm/clang we might want to see how a "regular Windows user" wants to use it, though. (not a problem for now)</div><div><br></div><div>symbolize_vs_style=true is the default on our port, but I can't open source it (yet?).</div><div><br></div><div>Thank you,</div><div><br></div><div> Filipe</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Thanks!<br>
<span class=""><br>
<br>
<a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_D7203&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=mQ4LZ2PUj9hpadE3cDHZnIdEwhEBrbAstXeMaFoB9tg&m=lDEwkSm5eQ7qeyWKaaRj1YOyKVsXtFiGTeouEKlBF5E&s=RXABxGMPDijkwrh5TsDsI9S1Io6-jpvJtWpLiCfrN4o&e=" rel="noreferrer" target="_blank">http://reviews.llvm.org/D7203</a><br>
<br>
Files:<br>
  lib/asan/asan_flags.inc<br>
  lib/asan/tests/asan_test_main.cc<br>
  test/asan/TestCases/Darwin/abort_on_error-darwin.cc<br>
  test/asan/TestCases/Linux/abort_on_error-linux.cc<br>
</span>  test/asan/Unit/<a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__lit.site.cfg.in&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=mQ4LZ2PUj9hpadE3cDHZnIdEwhEBrbAstXeMaFoB9tg&m=lDEwkSm5eQ7qeyWKaaRj1YOyKVsXtFiGTeouEKlBF5E&s=QdGuZvVOmjb3jelNGUeuEyIAyqmlrNEhJs7Q3wB4JLU&e=" rel="noreferrer" target="_blank">lit.site.cfg.in</a><br>
  test/asan/lit.cfg<br>
<br>
</blockquote></div><br></div></div>