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

Vedant Kumar via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jul 25 11:39:13 PDT 2017


vsk added a reviewer: vsk.
vsk added a comment.

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





More information about the cfe-commits mailing list