<br><br><div class="gmail_quote">On Wed, Nov 16, 2011 at 2:56 PM, Richard Smith <span dir="ltr"><<a href="mailto:richard@metafoo.co.uk">richard@metafoo.co.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
On Wed, November 16, 2011 02:40, Eli Friedman wrote:<br>
> 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. (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. <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>
> I would prefer -faddress-sanitizer over -fasan; shorter isn't really<br>
> better here.<br>
<br>
I think, more generally, we should have a naming strategy for options<br>
controlling runtime checks. For the IOC checks, it would be useful to have<br>
fine-grained control </blockquote><div>FYI</div><div>For AddressSanitizer we may also need fine-grained control in the future. </div><div>-faddress-sanitizer-[no-]reads</div><div>-faddress-sanitizer-[no-]stack</div><div>
-faddress-sanitizer-[no-]globals</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">over the individual checks for various reasons, so I<br>

suggest a -W-like mechanism, with individual runtime check flags, along with<br>
groups (eg all runtime overflow checks). As a straw man, we could use -R (e.g.<br>
-Raddress, -Roverflow, -Rno-shift-overflow) for this.<br>
<span class="HOEnZb"><font color="#888888">--<br>
Richard<br>
<br>
</font></span></blockquote></div><br>