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

Vedant Kumar via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 31 12:51:54 PDT 2017


vsk added a comment.

In https://reviews.llvm.org/D32842#826383, @shenhan wrote:

> Ping? Can we make a decision on this? 
>  I've this simple one D35849: [UBSan] Provide default blacklist filename for UBSan <https://reviews.llvm.org/D35849> which, depending on this, shall be discarded or move forward.
>  If this CL stalls, I'll seek to proceed with https://reviews.llvm.org/D35849. Any how, https://reviews.llvm.org/D35849 gets wiped out whence this CL (https://reviews.llvm.org/D32842) is submitted.


Please go ahead with https://reviews.llvm.org/D35849, I believe @pcc sgtm'd an identical patch of mine a while back, and it lgtm as well. Once I have time to revisit this I'll make sure to consider ubsan_blacklist.txt.


https://reviews.llvm.org/D32842





More information about the cfe-commits mailing list