Set the 'cached digest' flag after computing the digest for an

ImutAVLTree.  This was accidentally left out, and essentially caused
digest caching to be ignored in ImmutableMap and ImmutableSet (this
bug was detected from shark traces that showed ComputeDigest was in
the hot path in the clang static analyzer).

This reduces the running time of the clang static analyzer on an
example benchmark by ~32% for both RegionStore (field-sensitivty) and
BasicStore (without field-sensitivity).

llvm-svn: 80877
This commit is contained in:
Ted Kremenek 2009-09-03 04:21:34 +00:00
parent 11c989cc47
commit 231dd7169d
1 changed files with 1 additions and 0 deletions

View File

@ -331,6 +331,7 @@ private:
uint32_t X = ComputeDigest(getLeft(), getRight(), getValue());
Digest = X;
MarkedCachedDigest();
return X;
}
};