[PATCH] D30798: [analyzer] Turn suppress-c++-stdlib on by default

Anna Zaks via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Mar 11 10:17:28 PST 2017


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 invariants coming from libc++ headers and users have no ability to help the analyzer (ex: by inserting assertions). This results in a wave of false positives every time the headers change. Furthermore, we do not perform sufficient evaluation and testing for every new version of the headers.


Repository:
  rL LLVM

https://reviews.llvm.org/D30798





More information about the cfe-commits mailing list