Re: [PATCH] D24411: [Analyzer] Suppress false positives on the clang static analyzer

2016-09-19 Thread Anna Zaks via cfe-commits
zaks.anna added a comment. The thread from cfe-dev is called "Clang Static Analyzer: False Positive Suppression Support": http://clang-developers.42468.n3.nabble.com/Clang-Static-Analyzer-False-Positive-Suppression-Support-tt4053071.html https://reviews.llvm.org/D24411 __

Re: [PATCH] D24411: [Analyzer] Suppress false positives on the clang static analyzer

2016-09-19 Thread Nikhil Gupta via cfe-commits
nikhgupt added a comment. In https://reviews.llvm.org/D24411#545381, @zaks.anna wrote: > It is not clear to me that we've reached a consensus on cfe-dev list that > suppressing with comments and printing the checker name is the way to go. I'm new to the LLVM upstreaming process and have not be

Re: [PATCH] D24411: [Analyzer] Suppress false positives on the clang static analyzer

2016-09-16 Thread Anna Zaks via cfe-commits
zaks.anna added a comment. It is not clear to me that we've reached a consensus on cfe-dev list that suppressing with comments and printing the checker name is the way to go. https://reviews.llvm.org/D24411 ___ cfe-commits mailing list cfe-commits@

Re: [PATCH] D24411: [Analyzer] Suppress false positives on the clang static analyzer

2016-09-09 Thread Aditya Kumar via cfe-commits
hiraditya added a comment. IIRC, I was the primary author of this code, I'd really appreciate if you could attribute the code to the authors. Thanks https://reviews.llvm.org/D24411 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://list