[PATCH] D15225: [Driver] Sanitizer support based on runtime library presence

Kuba Brecka via cfe-commits cfe-commits at lists.llvm.org
Mon Dec 7 03:25:18 PST 2015


kubabrecka added a comment.

> The one thing I'm not sure about on this is, do we really want this to be darwin-only? I kinda think it might be nice if we unified this behavior across all platforms.

>  ... I wonder if this can't be better abstracted.

>  I'd really like to see all operating systems have the same behavior for this, so I think much of the code can be shared.

>  If the runtime library exists for a sanitizer (any sanitizer) I don't see any reason clang shouldn't then support the flags.


On Darwin, the situation is easier, because all the supported sanitizers are dylibs.  The code to link the runtimes is very different on every platform (Darwin, Linux, FreeBSD, Windows).  Alexey, can you comment on this?  Does this change make sense for Linux as well?


http://reviews.llvm.org/D15225





More information about the cfe-commits mailing list