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

Fedor Sergeev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 12 12:45:14 PDT 2018


fedor.sergeev added a comment.

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

> I think before.


Ok.

>>> 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?
> 
> I think you'll also need to do some deeper testing ... possibly by verifying BPI between each loop pass?

I'm afraid I do not fully get what kind of verification you are after.
But we can discuss it in that separate patch. Hopefully soon :)


Repository:
  rL LLVM

https://reviews.llvm.org/D43795





More information about the llvm-commits mailing list