[PATCH] D70376: [LVI] Restructure caching

[Nikita Popov via Phabricator via llvm-commits]

nikic updated this revision to Diff 232150.
nikic added a comment.

Don't use AssertingVH for BasicBlock. See D29006 <https://reviews.llvm.org/D29006> for why this needs to be a PoisoningVH. I believe using AssertingVH should be fine for the Value case though, as have a CallbackVH that should *always* remove these.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D70376/new/

https://reviews.llvm.org/D70376

Files:
  llvm/lib/Analysis/LazyValueInfo.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D70376.232150.patch
Type: text/x-patch
Size: 8387 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20191204/1341ea6b/attachment.bin>