[PATCH] D32842: Specify which sanitizers are covered by a sanitizer blacklist

Evgenii Stepanov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue May 30 13:23:46 PDT 2017


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





More information about the cfe-commits mailing list