[PATCH] D53076: [analyzer] WIP: Enhance ConditionBRVisitor to write out more information

[Csaba Dabis via Phabricator via cfe-commits]

Charusso created this revision.
Charusso added reviewers: NoQ, george.karpenkov.
Charusso added a project: clang.
Herald added subscribers: cfe-commits, donat.nagy, Szelethus, mikhail.ramalho, a.sidorin, szepet, eraman, xazax.hun.

I have noticed the `GDM` checking prevents to write out information of conditions. I tried to remove it, but then the `PostStmt` checking duplicated some message. I have not found a way to remove the duplication, so there is the two stage:

- first diff: removed `GDM` and `PostStmt` checking
- second diff: added the `PostStmt` visiting back to see what is duplicated


Repository:
  rC Clang

https://reviews.llvm.org/D53076

Files:
  lib/StaticAnalyzer/Core/BugReporterVisitors.cpp
  test/Analysis/Inputs/expected-plists/NewDelete-path-notes.cpp.plist
  test/Analysis/Inputs/expected-plists/cxx-for-range.cpp.plist
  test/Analysis/Inputs/expected-plists/edges-new.mm.plist
  test/Analysis/Inputs/expected-plists/inline-plist.c.plist
  test/Analysis/Inputs/expected-plists/model-file.cpp.plist
  test/Analysis/Inputs/expected-plists/plist-macros.cpp.plist
  test/Analysis/Inputs/expected-plists/plist-output.m.plist
  test/Analysis/Inputs/expected-plists/unix-fns.c.plist
  test/Analysis/MisusedMovedObject.cpp
  test/Analysis/NewDelete-path-notes.cpp
  test/Analysis/diagnostics/macros.cpp
  test/Analysis/diagnostics/no-store-func-path-notes.c
  test/Analysis/diagnostics/no-store-func-path-notes.cpp
  test/Analysis/diagnostics/no-store-func-path-notes.m
  test/Analysis/inline-plist.c
  test/Analysis/null-deref-path-notes.cpp
  test/Analysis/uninit-vals.m
  test/Analysis/virtualcall.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D53076.168990.patch
Type: text/x-patch
Size: 229704 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20181010/31ab8230/attachment-0001.bin>