<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div><div>On Nov 15, 2011, at 8:22 PM, Kostya Serebryany wrote:</div><blockquote type="cite"><div class="gmail_quote">On Tue, Nov 15, 2011 at 6:40 PM, Eli Friedman <span dir="ltr"><<a href="mailto:eli.friedman@gmail.com">eli.friedman@gmail.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="adm"><div id="q_133aa52cc00bbe31_0" class="ajR h4"><div class="ajT"></div></div></div><div class="h5">On Tue, Nov 15, 2011 at 6:22 PM, Kostya Serebryany <<a href="mailto:kcc@google.com">kcc@google.com</a>> wrote:<br>
> Hello,<br>
> Please review the following patch which adds -fasan (AddressSanitizer) flag<br>
> to clang.<br>
> (Mostly prepared by Chandler Carruth).<br>
> The LLVM part of AddressSanitizer has been submitted as r144758.<br>
> This patch will allow to enable AddressSanitizer from the clang command<br>
> line.<br>
> <a href="http://codereview.appspot.com/5396042" target="_blank">http://codereview.appspot.com/5396042</a><br>
> Coming next in separate patches:<br>
> - a patch to clang driver to pass linker flags for AddressSanitizer<br>
> (small)<br>
> - the run-time library and the tests (big)<br>
> Thanks,<br>
> --kcc<br>
<br>
</div></div>I would prefer -faddress-sanitizer over -fasan; shorter isn't really<br>
better here.<br></blockquote><div><br></div><div>Changing the flag name would sad.</div><div> - The logo of AddressSanitizer is a "Pheasant eating a Bug". "Fasan" means "Pheasant" in several European languages (also, "Фазан" in Russian).</div>
<div>- Quite a few existing users will have to change their makefiles. </div></div></blockquote><div><br></div><div>Them's the breaks of early adoption.</div><div><br></div><div>-fasan is not an acceptable name, no matter how nice the pun. And I do appreciate the pun.</div><div><br></div><div>John.</div></div></body></html>