diff options
author | Ted Kremenek <kremenek@apple.com> | 2012-09-07 22:31:01 +0000 |
---|---|---|
committer | Ted Kremenek <kremenek@apple.com> | 2012-09-07 22:31:01 +0000 |
commit | 244e1d7d0f2873c37c200988f32b081552c2d51f (patch) | |
tree | a4b73b07cb9fd3ceb94fba780b9e9a09a14aae74 /clang/lib/StaticAnalyzer/Core/CheckerRegistry.cpp | |
parent | 334ad6ac13b6a43041fc52f267cf4ab9d4861879 (diff) | |
download | bcm5719-llvm-244e1d7d0f2873c37c200988f32b081552c2d51f.tar.gz bcm5719-llvm-244e1d7d0f2873c37c200988f32b081552c2d51f.zip |
Remove ProgramState::getSymVal(). It was being misused by Checkers,
with at least one subtle bug in MacOSXKeyChainAPIChecker where the
calling the method was a substitute for assuming a symbolic value
was null (which is not the case).
We still keep ConstraintManager::getSymVal(), but we use that as
an optimization in SValBuilder and ProgramState::getSVal() to
constant-fold SVals. This is only if the ConstraintManager can
provide us with that information, which is no longer a requirement.
As part of this, introduce a default implementation of
ConstraintManager::getSymVal() which returns null.
For Checkers, introduce ConstraintManager::isNull(), which queries
the state to see if the symbolic value is constrained to be a null
value. It does this without assuming it has been implicitly constant
folded.
llvm-svn: 163428
Diffstat (limited to 'clang/lib/StaticAnalyzer/Core/CheckerRegistry.cpp')
0 files changed, 0 insertions, 0 deletions