[PATCH] D30790: [LVI] Add an LVI printer pass to capture test LVI cache after transformations

[Anna Thomas via Phabricator via llvm-commits]

anna marked 2 inline comments as done.
anna added inline comments.


================
Comment at: include/llvm/Analysis/LazyValueInfo.h:102
+  // Print the LVI cache.
+  void printLazyValueInfoCache(const DataLayout &DL, raw_ostream &OS);
+
----------------
apilipenko wrote:
> Here and below - just printCache?
how about `printCaches`? Since we'll print both the overdefined and the LVICache.


================
Comment at: lib/Analysis/LazyValueInfo.cpp:449
+             << BV.first->getName() << "'\n";
+    }
+
----------------
apilipenko wrote:
> For the sake of completeness you might also want to print overdefined cache. 
agreed. I've added printing of overdefined per basic block.


https://reviews.llvm.org/D30790