[PATCH] D34000: [LazyValueInfo] Don't run the more complex predicate handling code for EQ and NE in getPredicateResult

Anna Thomas via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 9 06:53:22 PDT 2017


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

In https://reviews.llvm.org/D34000#776491, @craig.topper wrote:

> This function is just calculating a predicate result based on a LVI lattice value. I don't think it affects the calculation of LVI lattice values themselves. So I don't think the printer can see it.


agreed, `getPredicateAt` uses the LVI result within CVP and Jumpthreading.

LGTM.


https://reviews.llvm.org/D34000





More information about the llvm-commits mailing list