summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPeter Collingbourne <peter@pcc.me.uk>2017-02-15 03:50:01 +0000
committerPeter Collingbourne <peter@pcc.me.uk>2017-02-15 03:50:01 +0000
commit96e36a67ed98340da023595dfc173340d74ecfbb (patch)
tree87256f04113d435123cd497f5004d901294722df
parent650e5b28d2472eb93c210b6a6275afa95ce98bdb (diff)
downloadbcm5719-llvm-96e36a67ed98340da023595dfc173340d74ecfbb.tar.gz
bcm5719-llvm-96e36a67ed98340da023595dfc173340d74ecfbb.zip
AssumptionCache: Update documentation comment.
The comment was somewhat misleading in that it implied that passes were not responsible for adding new assumptions to the assumption cache. This new wording now explicitly mentions that they are required to do so. Differential Revision: https://reviews.llvm.org/D29977 llvm-svn: 295148
-rw-r--r--llvm/include/llvm/Analysis/AssumptionCache.h9
1 files changed, 4 insertions, 5 deletions
diff --git a/llvm/include/llvm/Analysis/AssumptionCache.h b/llvm/include/llvm/Analysis/AssumptionCache.h
index 72c67955a95..bd29292c4c6 100644
--- a/llvm/include/llvm/Analysis/AssumptionCache.h
+++ b/llvm/include/llvm/Analysis/AssumptionCache.h
@@ -31,11 +31,10 @@ namespace llvm {
/// \brief A cache of @llvm.assume calls within a function.
///
/// This cache provides fast lookup of assumptions within a function by caching
-/// them and amortizing the cost of scanning for them across all queries. The
-/// cache is also conservatively self-updating so that it will never return
-/// incorrect results about a function even as the function is being mutated.
-/// However, flushing the cache and rebuilding it (or explicitly updating it)
-/// may allow it to discover new assumptions.
+/// them and amortizing the cost of scanning for them across all queries. Passes
+/// that create new assumptions are required to call registerAssumption() to
+/// register any new @llvm.assume calls that they create. Deletions of
+/// @llvm.assume calls do not require special handling.
class AssumptionCache {
/// \brief The function for which this cache is handling assumptions.
///
OpenPOWER on IntegriCloud