vsk abandoned this revision.
vsk added a comment.
Obsoleted by: https://reviews.llvm.org/D32842
https://reviews.llvm.org/D32043
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
vsk added a comment.
Ok. I will take a step back and see what it will take to implement
per-sanitizer blacklists.
https://reviews.llvm.org/D32043
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo
eugenis added a comment.
We definitely want different blacklists for ASan and MSan.
https://reviews.llvm.org/D32043
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
vsk added a comment.
In https://reviews.llvm.org/D32043#728427, @pcc wrote:
> This seems reasonable to me, although it's unfortunate that the design of the
> sanitizer blacklist feature does not (at present) allow different blacklists
> for different sanitizers.
IMO this might be a real probl
eugenis added a comment.
Looks fine.
https://reviews.llvm.org/D32043
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
pcc added a comment.
This seems reasonable to me, although it's unfortunate that the design of the
sanitizer blacklist feature does not (at present) allow different blacklists
for different sanitizers.
@eugenis what do you think?
https://reviews.llvm.org/D32043
vsk created this revision.
This patch removes a limitation which causes us to load at most one
default sanitizer blacklist when multiple sanitizers are enabled. E.g if
asan + cfi are enabled, and default blacklists for both sanitizers are
present, we would only load one of the blacklists.
The new
