<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Jul 22, 2015 at 1:24 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:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">kcc added a comment.<br>
<br>
Ok. You seem to be serious about this.<br>
Let us<br>
<br>
- Set asan_abort_on_error=(SANITIZER_MAC | ...) as you have it now<br>
- Update config.environment['ASAN_OPTIONS'] but only under "if OSX"<br>
- Same for asan_test_main.cc<br>
<br>
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>
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></blockquote><div><br></div><div>Sounds good. I can take it out and keep that line private on our side, for now, conditional on ps4 (or other platforms which default to VS style) as a target.</div><div>Actually… Kuba: Can you remove that default and add a "if <we're targetting a mac-like platform>: config.environment['ASAN_OPTIONS'] ='abort_on_error=true'" when you land this patch?</div><div><br></div><div>Thanks,</div><div><br></div><div>  Filipe</div><div> </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">================<br>
Comment at: test/asan/TestCases/Darwin/abort_on_error-darwin.cc:7<br>
@@ +6,3 @@<br>
<span class="">+// Intentionally don't inherit the default ASAN_OPTIONS.<br>
+// RUN: ASAN_OPTIONS="" not --crash %run %t 2>&1 | FileCheck %s<br>
+<br>
</span>----------------<br>
add a line w/o resetting ASAN_OPTIONS. On mac it will exit() because of the change in lit.cfg. We want to still test for it so that we don't run all tests with abort_on_error.<br>
Update the comments.<br>
<br>
================<br>
Comment at: test/asan/TestCases/Linux/abort_on_error-linux.cc:7<br>
@@ +6,3 @@<br>
<span class="">+// Intentionally don't inherit the default ASAN_OPTIONS.<br>
+// RUN: ASAN_OPTIONS="" not run %t 2>&1 | FileCheck %s<br>
+<br>
</span>----------------<br>
add a line w/o ASAN_OPTIONS and update the comment.<br>
We don't want to set ASAN_OPTIONS on Linux in lit.cfg<br>
<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=bFq9XcA293HjWbg-rr1lFZVRtN6kiUgJvPjWcOo6alc&s=EkXkPHHKlZdzT__-SGDisWkbVc6I1nxPXqlXmU7Ge2s&e=" rel="noreferrer" target="_blank">http://reviews.llvm.org/D7203</a><br>
<br>
<br>
<br>
</blockquote></div><br></div></div>