On Wed, Dec 5, 2012 at 5:41 AM, Evgeniy Stepanov <span dir="ltr"><<a href="mailto:eugenis@google.com" target="_blank">eugenis@google.com</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div style="font-family:arial,helvetica,sans-serif;font-size:10pt"><div dir="ltr">Done and committed. I also noticed that we do not catch conflicts between memory and other sanitizers.<div><br></div><div>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.</div>
</div></div></blockquote><div><br></div><div>Here's what we did when UBSan couldn't be combined with either TSan or ASan:</div><div><br></div><div><a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/Tools.cpp?r1=168958&r2=169066">http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/Tools.cpp?r1=168958&r2=169066</a></div>
<div><br></div><div>... but this seems fine as it is now.</div></div>