[llvm] 7adb9e0 - [LoopLoadElim] Add test showing that LoopLoadElim doesn't work correctly with new PM

Florian Hahn via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 8 03:41:53 PDT 2020



> On Apr 8, 2020, at 11:32, Max Kazantsev via llvm-commits <llvm-commits at lists.llvm.org> wrote:
> 
> 
> Author: Max Kazantsev
> Date: 2020-04-08T17:32:03+07:00
> New Revision: 7adb9e06fd0dd49891e491cba9705e9dd119440e
> 
> URL: https://github.com/llvm/llvm-project/commit/7adb9e06fd0dd49891e491cba9705e9dd119440e
> DIFF: https://github.com/llvm/llvm-project/commit/7adb9e06fd0dd49891e491cba9705e9dd119440e.diff
> 
> LOG: [LoopLoadElim] Add test showing that LoopLoadElim doesn't work correctly with new PM

The problem might be that LoopSimplify is not run automatically with the new pass manager, but is with the old one. It seems like we have to manually run loopSimplify with the new pass manager, e.g. as IRCE already does: https://github.com/llvm/llvm-project/blob/master/llvm/lib/Transforms/Scalar/InductiveRangeCheckElimination.cpp#L1814


More information about the llvm-commits mailing list