Alina Sbirlea via llvm-dev
2020-Jan-28 22:44 UTC
[llvm-dev] RFC: Changing Inductive Range Check Elimination to a Function Pass
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 <https://reviews.llvm.org/D72891>). Making IRCE a FunctionPass will address this issue. Thanks, Alina -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20200128/b18437cf/attachment.html>
Alina Sbirlea via llvm-dev
2020-Jan-29 00:20 UTC
[llvm-dev] RFC: Changing Inductive Range Check Elimination to a Function Pass
To give an idea of the changes, I uploaded: https://reviews.llvm.org/D73592 . Thanks, Alina On Tue, Jan 28, 2020 at 2:44 PM Alina Sbirlea <alina.sbirlea at gmail.com> wrote:> 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 <https://reviews.llvm.org/D72891>). Making IRCE a > FunctionPass will address this issue. > > Thanks, > Alina >-------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20200128/138aa9b8/attachment.html>