[PATCH] D26759: Remove unused check::RegionChanges::wantsRegionChangeUpdate callback

2016-11-16 Thread Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL287175: [analyzer] Remove unused check::RegionChanges::wantsRegionChangeUpdate callback (authored by zaks). Changed prior to commit: https://reviews.llvm.org/D26759?vs=78249=78271#toc Repository: rL

[PATCH] D26759: Remove unused check::RegionChanges::wantsRegionChangeUpdate callback

2016-11-16 Thread Krzysztof Wiśniewski via cfe-commits
k-wisniewski updated this revision to Diff 78249. k-wisniewski added a comment. Removed a line that leaked from another diff - thanks Anna! https://reviews.llvm.org/D26759 Files: include/clang/StaticAnalyzer/Core/Checker.h include/clang/StaticAnalyzer/Core/CheckerManager.h

[PATCH] D26759: Remove unused check::RegionChanges::wantsRegionChangeUpdate callback

2016-11-16 Thread Anna Zaks via cfe-commits
zaks.anna added inline comments. Comment at: lib/StaticAnalyzer/Core/CheckerManager.cpp:535 + ExplicitRegions, Regions, + Call, LCtx); } Looks like the other patch leaked

[PATCH] D26759: Remove unused check::RegionChanges::wantsRegionChangeUpdate callback

2016-11-16 Thread Anna Zaks via cfe-commits
zaks.anna accepted this revision. zaks.anna added a comment. This revision is now accepted and ready to land. Looks great! https://reviews.llvm.org/D26759 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D26759: Remove unused check::RegionChanges::wantsRegionChangeUpdate callback

2016-11-16 Thread Krzysztof Wiśniewski via cfe-commits
k-wisniewski created this revision. k-wisniewski added reviewers: NoQ, dcoughlin, zaks.anna, a.sidorin. k-wisniewski added a subscriber: cfe-commits. Hi! I've noticed that check::RegionChanges::wantsRegionChangeUpdate is no longer used - as far as the discussion went it has been unused at least