[PATCH] D62978: [analyzer] ReturnVisitor: Handle unknown ReturnStmts better

Csaba Dabis via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 10 12:59:22 PDT 2019


Charusso added a comment.

In D62978#1533558 <https://reviews.llvm.org/D62978#1533558>, @NoQ wrote:

> Ah, that positive!


Positive == true positive, not false positive, I got it.

> No, i don't think this is a valid way to suppress it.

Bought me, they are worth to report. The misleading reports made me think I have to suppress them.

Example (difference starts at note 33, at line 410):
F9159208: before.html <https://reviews.llvm.org/F9159208>
F9159209: after.html <https://reviews.llvm.org/F9159209>


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

https://reviews.llvm.org/D62978





More information about the cfe-commits mailing list