<div dir="ltr"><div>Hi all,</div><div>Sorry for not replying earlier.</div><div><br></div>Kostya:<div>It encourages changes if needed, yes.<div>Like when a platform needs to change something related to other platforms where ASan runs, and would require lots of changes to tests.</div><div><br></div><div>We also clear away ASAN_OPTIONS (and the others) when we run tests, which allows us to make minimum assumptions when running them. This specific change seems to be strictly on the "positive" side. Without it we could simply fail some tests because the user happened to have a specific option turned on by default and ran the ASan tests.</div><div><br></div><div>By having ASAN_OPTIONS be reset by lit, and then having each test that cares about it adding or replacing it, we can test the options themselves, their absence, and have "proper" defaults for the tests.</div><div>It also allows us to "easily" have platforms specify their own defaults for flags, like what Kuba wants to do.<br></div><div><br></div><div>About this specific change:</div><div>It's surprising (to me) to have ASan, by default, exit() the process, as if "no error" had happened (by which I mean: The process exited in a "normal" way, even though it exited with a failure exit code). This doesn't affect me much, as an exit() on my platform is also "weird". But on Mac OS X it feels weird to not have it crash.</div><div><br></div><div><br></div><div>Regards,</div><div><br></div><div>  Filipe</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jul 21, 2015 at 3:30 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>
But the existing state encourage more changes to ASAN_OPTIONS in lit.cfg.<br>
We at the very least should warn in the comments about adding more options there,<br>
but better remove this too.<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=zPMWn7XpB8zBvJ4NwAXYvrlMukmEYdl5UgClY5Sw-C8&s=6x3i9mAtkxYBfMzUXtABxBRZY3EgPGySLn-H2DIYmQY&e=" rel="noreferrer" target="_blank">http://reviews.llvm.org/D7203</a><br>
<br>
<br>
<br>
</blockquote></div><br></div></div></div>