[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