zaks.anna added a comment.
Correct, this will suppress some valid warnings that occur due to user errors
and valid warnings coming from the standard library.
However, I believe this is the right choice right now since we know that the
analyzer is not currently effective in understanding invaria
a.sidorin added a comment.
I have no any objection on this change.
Repository:
rL LLVM
https://reviews.llvm.org/D30798
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
NoQ added a comment.
I'd like to clarify that in case of path-sensitive analysis there are actually
three warning classes to consider.
1. Warnings that reside completely in system headers and indicate bugs in
system headers, most likely falsely.
2. Warnings that originate from the main file and
xazax.hun added a comment.
In https://reviews.llvm.org/D30798#697115, @zaks.anna wrote:
> I've committed the change, but would very much appreciate community feedback
> here if if there is any!
I agree with the change. Users are usually not interested in the results from
the standard library,
zaks.anna added a comment.
I've committed the change, but would very much appreciate community feedback
here if if there is any!
Repository:
rL LLVM
https://reviews.llvm.org/D30798
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://l
This revision was automatically updated to reflect the committed changes.
Closed by commit rL297429: [analyzer] Turn suppress-c++-stdlib on by default
(authored by zaks).
Changed prior to commit:
https://reviews.llvm.org/D30798?vs=91236&id=91238#toc
Repository:
rL LLVM
https://reviews.llvm.
dcoughlin accepted this revision.
dcoughlin added a comment.
This revision is now accepted and ready to land.
LGTM,
https://reviews.llvm.org/D30798
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinf
zaks.anna created this revision.
We have several reports of false positives coming from libc++. For example,
there are reports of false positives in std::regex, std::wcout, and also a
bunch of issues are reported in https://reviews.llvm.org/D30593. In many cases,
the analyzer trips over the com
