[PATCH] D35849: [UBSan] Provide default blacklist filename for UBSan

[Han Shen via Phabricator via cfe-commits]

shenhan added a comment.

Thanks for pointing out the relevant CLs. I agree that's the clearer and better solution.

(The only concern is that it has been sitting there for 2 months. I'll check back.)

In https://reviews.llvm.org/D35849#820473, @vsk wrote:

> This won't do the right thing if more than one sanitizer with a default blacklist is enabled. It's also problematic that a default blacklist for one sanitizer can blacklist code for a different sanitizer. See:
>
> https://reviews.llvm.org/D32043
>  https://reviews.llvm.org/D32047
>  https://reviews.llvm.org/D32842
>
> IMO I think https://reviews.llvm.org/D32842 is the right path forward. I don't have the time to get back to it this week, but if you'd like to pick it up and respond to review feedback from @eugenis, please do (I can certainly help with the review).





https://reviews.llvm.org/D35849