[PATCH] D43795: [New PM][IRCE] port of Inductive Range Check Elimination pass to the new pass manager

[Fedor Sergeev via Phabricator via llvm-commits]

fedor.sergeev added a comment.

In https://reviews.llvm.org/D43795#1035151, @chandlerc wrote:

> I would just suggest making BPI required as part of the loop standard analyses.
>  It should be a separate patch (with clear discussion pointing out why it is useful), but BPI is useful to *many* loop passes I suspect, I just think we hadn't wired it up previously.


Should this separate BPI patch go before or after this change?

> As part of this we should do some basic testing to make sure that the loop passes preserve BPI correctly.

Checking -debug-pass-manager output for "Running analysis:" and  absence of "Invalidating analysis:", right?


Repository:
  rL LLVM

https://reviews.llvm.org/D43795