[PATCH] D29715: LVI: Add a per-value worklist limit to LazyValueInfo.

[Daniel Jasper via Phabricator via llvm-commits]

djasper added inline comments.


================
Comment at: lib/Analysis/LazyValueInfo.cpp:659
+        std::pair<BasicBlock *, Value *> &e = BlockValueStack.top();
+        TheCache.insertResult(e.second, e.first,
+                              LVILatticeVal::getOverdefined());
----------------
Wouldn't this lead to potentially adding Blocks to the cache where we haven't done any search (i.e. that got added after 499 processed values)? Is that a problem?


https://reviews.llvm.org/D29715