[cfe-commits] [PATCH] Add -fsanitize= command line argument to control -fcatch-undefined-behavior, -faddress-sanitizer, and -fthread-sanitizer

Matthieu Monrocq matthieu.monrocq at gmail.com
Fri Nov 2 11:06:45 PDT 2012


On Fri, Nov 2, 2012 at 2:11 PM, Dmitry Vyukov <dvyukov at google.com> wrote:

> On Fri, Nov 2, 2012 at 4:53 PM, Diego Novillo <dnovillo at google.com> wrote:
>
>> On Fri, Nov 2, 2012 at 2:01 AM, Dmitry Vyukov <dvyukov at google.com> wrote:
>>
>> > +Diego,David
>> >
>> > And if/when we add MemorySanitizer, can we add new alias
>> -fmemory-sanitizer?
>> > I am asking because we are in the process of committing asan/tsan to
>> gcc,
>> > and we want to preserve consistency between clang and gcc. Gcc uses
>> > -address/thread-sanitizer.
>>
>> I agree with Chandler.  Using compatible flags in both compiler is
>> going to be the best option for users.  I only ask that we don't start
>> changing these flags every so often (unless there is a very good
>> reason for doing so).
>>
>
>
> So what naming scheme do we choose as main?
> If it's -sanitizer=xxx, then gcc must be changed.
>
>
>
Just to make sure everyone is on the same basis, Richard's proposal is to
have the flag be  `-fsanitize` => Note the leading `f` and the absence of
trailing `r`.

-- Matthieu
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20121102/4e0ce335/attachment.html>


More information about the cfe-commits mailing list