[cfe-commits] [PATCH] Clang support for MemorySanitizer

Richard Smith richard at metafoo.co.uk
Wed Dec 5 15:25:05 PST 2012


On Wed, Dec 5, 2012 at 5:41 AM, Evgeniy Stepanov <eugenis at google.com> wrote:

> Done and committed. I also noticed that we do not catch conflicts between
> memory and other sanitizers.
>
> Let me know if you are not comfortable with a bit of code duplication
> there. I think that N*(N-1) explicit checks (for N=3) is still better and
> more readable than generic code for arbitrary number of sanitizers.
>

Here's what we did when UBSan couldn't be combined with either TSan or ASan:

http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/Tools.cpp?r1=168958&r2=169066

... but this seems fine as it is now.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20121205/0c19b06c/attachment.html>


More information about the cfe-commits mailing list