RFC: Changing Inductive Range Check Elimination to a Function Pass

Project Infrastructure LLVM Dev List Archives
#1

Hi,

Recently the LoopUnrollAndJamPass was made a function pass, and here was the motivation for that: https://groups.google.com/forum/#!topic/llvm-dev/LF4rUjkVI2g.
I don’t see an obvious reason to keep IRCE as a LoopPass either, so I thought folks familiar with the pass could chime in if this seems like the right change.
An additional motivation is the need to either preserve BranchProbabilityInfo or have a new instance computed for each Loop (D72891). Making IRCE a FunctionPass will address this issue.

Thanks,
Alina

#2

To give an idea of the changes, I uploaded: https://reviews.llvm.org/D73592.

Thanks,
Alina