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

[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 since checkPointerEscape has been added. This patch removes unused code as a part of cleanup/improvement in check::RegionChanges that I have undertaken. Let's discuss the necessity of such action here and decide upon it asap, because I need to know what to do with my another patch that adds LocationContext (or CheckerContext) to check::RegionChanges::checkRegionChanges.


https://reviews.llvm.org/D26759

Files:
  include/clang/StaticAnalyzer/Core/Checker.h
  include/clang/StaticAnalyzer/Core/CheckerManager.h
  include/clang/StaticAnalyzer/Core/PathSensitive/ExprEngine.h
  include/clang/StaticAnalyzer/Core/PathSensitive/SubEngine.h
  lib/StaticAnalyzer/Checkers/CStringChecker.cpp
  lib/StaticAnalyzer/Checkers/RetainCountChecker.cpp
  lib/StaticAnalyzer/Core/CheckerManager.cpp
  lib/StaticAnalyzer/Core/ExprEngine.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D26759.78234.patch
Type: text/x-patch
Size: 8469 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20161116/5abe7b38/attachment.bin>