<div dir="ltr">I think that making blacklist available in Clang is a good idea.<div style>I agree with Richard's comment as well.</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jan 17, 2013 at 12:50 AM, Richard Smith <span dir="ltr"><<a href="mailto:richard@metafoo.co.uk" target="_blank">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 the clang side, it seems messy and error-prone to distribute the<br>
checking for blacklisting across all the callers of EmitCheck, but I<br>
understand that you're trying to avoid generating the check condition<br>
for blacklisted functions. Maybe we could make a separate struct<br>
containing just the Sanitize* flags, and in the CodeGenFunction<br>
constructor bind a reference to either the global set of flags or a<br>
set of all-false flags, depending on whether we're blacklisted? This<br>
would also make it easier to blacklist specific sanitizers in specific<br>
functions later.<br>
<div><div class="h5"><br>
On Wed, Jan 16, 2013 at 12:08 PM, Will Dietz <<a href="mailto:wdietz2@illinois.edu">wdietz2@illinois.edu</a>> wrote:<br>
> Minor touchup to clang patch, thanks!<br>
><br>
> ~Will<br>
><br>
> On Wed, Jan 16, 2013 at 11:37 AM, Will Dietz <<a href="mailto:wdietz2@illinois.edu">wdietz2@illinois.edu</a>> wrote:<br>
>> Attached are two patches.<br>
>><br>
>> First applies to clang and uses the blacklist to avoid instrumenting<br>
>> the source files or functions specified.  Lit test included.<br>
>><br>
>> The other is a small change to llvm to make the "Blacklist" class<br>
>> visible to Clang.  Not sure I understand the header organization<br>
>> system well enough, let me know if it belongs elsewhere.<br>
>><br>
>> Thanks!<br>
>><br>
>> ~Will<br>
><br>
</div></div>> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
><br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div>Alexey Samsonov, MSK</div>
</div>