Skip to content

Commit fd1ff5e

Browse files
committed
[Inliner] Make ModuleInlinerWrapperPass return PreservedAnalyses::all()
The ModulePassManager should already have taken care of all analysis invalidation. Without this change, upcoming changes will cause more invalidation than necessary. Reviewed By: mtrofin Differential Revision: https://reviews.llvm.org/D101320
1 parent c6d5e6b commit fd1ff5e

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

llvm/lib/Transforms/IPO/Inliner.cpp

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1036,8 +1036,10 @@ PreservedAnalyses ModuleInlinerWrapperPass::run(Module &M,
10361036
else
10371037
MPM.addPass(createModuleToPostOrderCGSCCPassAdaptor(
10381038
createDevirtSCCRepeatedPass(std::move(PM), MaxDevirtIterations)));
1039-
auto Ret = MPM.run(M, MAM);
1039+
MPM.run(M, MAM);
10401040

10411041
IAA.clear();
1042-
return Ret;
1042+
1043+
// The ModulePassManager has already taken care of invalidating analyses.
1044+
return PreservedAnalyses::all();
10431045
}

0 commit comments

Comments
 (0)