eugenis added a comment.

This change scares me a bit, to be honest. Is this really that big of a 
problem? Blacklists are not supposed to be big, maybe we can tolerate a few 
false negatives?

Consider extending the blacklist syntax instead, the same way Valgrind does. 
Blacklist entries could be annotated with a list of sanitizers they apply to, 
like

asan,ubsan : src: file.cc:line

or an even less verbose way using sections

[asan]
src:
src:
[msan]
src:

As an extra benefit, all default blacklists can be merged into one.


https://reviews.llvm.org/D32842



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to