Charusso added a comment.

In D66042#1626122 <https://reviews.llvm.org/D66042#1626122>, @NoQ wrote:

> > use it locally
>
> What do you mean here? If you want to use the patch for evaluating your 
> results, you might as well untick the checker in the scan-build's index.html 
> interface. The point of having this patch landed is to allow users who are 
> worried by false positives of specific core checkers to use the Static 
> Analyzer on their code anyway, without being overwhelmed with false 
> positives. It was never about us, it was always about them^^


Well, it is not that difficult to write up a documentation: apply that patch so 
LLVM reports will be more awesome and we have not got enough time to make this 
large-scale tough change upstreamed:

  1  // See whether we need to silence the checker.
  2  StringRef ErrorTag = 
ErrorNode->getLocation().getTag()->getTagDescription();
  3  for (const std::string &CheckerName : Opts.CheckerSilenceVector)
  4    if (ErrorTag.startswith(CheckerName))
  5      return nullptr;


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D66042/new/

https://reviews.llvm.org/D66042



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to