vitalybuka wrote:

We should revert this patch. It breaks mupltiple bots, introducing Msan reports.

Also on a first look I don't like `-fsanitize-pattern-exclusion=` at all, very 
inconsistent.
I assumed we agreed to splitting sanitizers into smaller sets?
I would like to know why we need to introduce a new approach instead of using 
existing one.

Sorry, but I am not sure why this didn't show up my 
https://github.com/llvm/llvm-project/pulls?q=is%3Aopen+is%3Apr+review-requested%3A%40me+sort%3Aupdated-desc
 I wanted to review this patch.

@MaskRay maybe have opinion here as well?

https://github.com/llvm/llvm-project/pull/100272
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to