Looks good in general, I'll commit later unless someone objects.<div><br></div><div>Just curious, what is your use case? </div><div>We sometimes use the blacklists for asan/msan/tsan during initial deployment in a particular source base,</div>
<div>but long term we always try to fix the code (and mark the tricky code with attributes) and get rid of the blacklist. </div><div><br></div><div>Thanks! </div><div><br></div><div>--kcc </div><div><br><br><div class="gmail_quote">
On Wed, Aug 22, 2012 at 11:00 PM, Reid Watson <span dir="ltr"><<a href="mailto:reidw@google.com" target="_blank">reidw@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Also, for ease of review, here's a version with no renaming of files<br>
involved.  It preserves the FunctionBlackList name, though that should<br>
probably be changed if this is committed.<br>
<div class="HOEnZb"><div class="h5"><br>
On Wed, Aug 22, 2012 at 11:08 AM, Reid Watson <<a href="mailto:reidw@google.com">reidw@google.com</a>> wrote:<br>
> Hi all,<br>
><br>
> First off, sincere thanks to reviewers(kcc especially) for helping<br>
> with the initial initialization order patch.<br>
><br>
> I'm now looking to expand the blacklisting functionality of<br>
> AddressSanitizer to include the ability to blacklist global variables.<br>
>  This is currently handled in FunctionBlackList.h, which is renamed to<br>
> BlackList.h to reflect the fact that it can store more than functions.<br>
><br>
> I've attached the patch and uploaded it to<br>
> Rietveld(<a href="http://codereview.appspot.com/6473048/" target="_blank">http://codereview.appspot.com/6473048/</a>).  Review is<br>
> appreciated as always.<br>
><br>
> All the best,<br>
> Reid<br>
</div></div></blockquote></div><br></div>