[PATCH] D32135: [LVI Printer] Rely on the LVI analysis functions rather than the LVI cache

[Anna Thomas via Phabricator via llvm-commits]

anna created this revision.

LVIPrinter pass was previously relying on the LVICache. We now directly call the
the LVI functions which solves the value if the LVI information is not already
available in the cache. This has 2 benefits over the printing of LVI cache:

1. higher coverage (i.e. catches errors) in LVI code when cache value is

invalidated.

2. relies on the core functions, and not dependent on the LVI cache (which may

be scrapped at some point).
It would still catch any cache invalidation errors, since we first go through
the cache.


https://reviews.llvm.org/D32135

Files:
  include/llvm/Analysis/LazyValueInfo.h
  lib/Analysis/LazyValueInfo.cpp
  test/Analysis/LazyValueAnalysis/lvi-after-jumpthreading.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D32135.95481.patch
Type: text/x-patch
Size: 12301 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170417/e5cefd65/attachment.bin>