[PATCH] D70376: [LVI] Restructure caching

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 13 10:32:59 PDT 2020


fhahn added a comment.

In D70376#2145969 <https://reviews.llvm.org/D70376#2145969>, @nikic wrote:

> Ideally we'd switch LVI to be based on PredicateInfo and thus avoid the need to track data per-block in the first place. Of course, this is not simple to do.


Alternatively we might be able to move some of the logic from LVI/CVP to (IP)SCCP, which already uses PredicateInfo. At the moment, there is already a bit of function overlap, but currently SCCP mostly uses ranges to simplify conditions, but could also do more things.

There probably are a few things in LVI/CVP that might not be possible to handle in (IP)SCCP, e.g. I am not sure about the edge threading.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70376





More information about the llvm-commits mailing list