[PATCH] D59349: Let CorrelatedValuePropagation preserve LazyValueInfo

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 26 18:13:23 PDT 2019


reames accepted this revision.
reames added a comment.
This revision is now accepted and ready to land.

LGTM.  This may be slightly imprecise - for instance, when we fold an operand which allows further LVI analysis - but the preservation is conservatively correct.

p.s. Updating a review resets the status.


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

https://reviews.llvm.org/D59349





More information about the llvm-commits mailing list