[PATCH] D27090: Add LocationContext as a parameter to checkRegionChanges

2017-01-12 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL291869: [analyzer] Add LocationContext as a parameter to checkRegionChanges (authored by zaks). Changed prior to commit: https://reviews.llvm.org/D27090?vs=79204&id=84197#toc Repository: rL LLVM htt

[PATCH] D27090: Add LocationContext as a parameter to checkRegionChanges

2017-01-12 Thread Anna Zaks via Phabricator via cfe-commits
zaks.anna accepted this revision. zaks.anna added a comment. This revision is now accepted and ready to land. This needs to be committed. https://reviews.llvm.org/D27090 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/c

[PATCH] D27090: Add LocationContext as a parameter to checkRegionChanges

2016-11-30 Thread Anna Zaks via Phabricator via cfe-commits
zaks.anna added a comment. Thank you for doing this!!! Artem, do you want to disallow the creation of successors from checkRegionChanges callback or can this go in as is? Anna. Comment at: lib/StaticAnalyzer/Checkers/CheckerDocumentation.cpp:240 - /// #checkRegionChanges wh

[PATCH] D27090: Add LocationContext as a parameter to checkRegionChanges

2016-11-24 Thread Krzysztof Wiśniewski via cfe-commits
k-wisniewski created this revision. k-wisniewski added reviewers: zaks.anna, dcoughlin, NoQ, a.sidorin. k-wisniewski added subscribers: cfe-commits, zaks.anna. This patch adds LocationContext to checkRegionChanges. This patch has been distilled out from other changes that I'll add separately. Fol