From fd1ff5ee0388af9eb373d903a1644e6cd2c96d9d Mon Sep 17 00:00:00 2001 From: Arthur Eubanks Date: Mon, 26 Apr 2021 12:53:09 -0700 Subject: [PATCH] [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 --- llvm/lib/Transforms/IPO/Inliner.cpp | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/llvm/lib/Transforms/IPO/Inliner.cpp b/llvm/lib/Transforms/IPO/Inliner.cpp index caba83b..1c9800f 100644 --- a/llvm/lib/Transforms/IPO/Inliner.cpp +++ b/llvm/lib/Transforms/IPO/Inliner.cpp @@ -1036,8 +1036,10 @@ PreservedAnalyses ModuleInlinerWrapperPass::run(Module &M, else MPM.addPass(createModuleToPostOrderCGSCCPassAdaptor( createDevirtSCCRepeatedPass(std::move(PM), MaxDevirtIterations))); - auto Ret = MPM.run(M, MAM); + MPM.run(M, MAM); IAA.clear(); - return Ret; + + // The ModulePassManager has already taken care of invalidating analyses. + return PreservedAnalyses::all(); } -- 2.7.4