[llvm] fd1ff5e - [Inliner] Make ModuleInlinerWrapperPass return PreservedAnalyses::all()

[Arthur Eubanks via llvm-commits]

Author: Arthur Eubanks
Date: 2021-04-26T17:22:35-07:00
New Revision: fd1ff5ee0388af9eb373d903a1644e6cd2c96d9d

URL: https://github.com/llvm/llvm-project/commit/fd1ff5ee0388af9eb373d903a1644e6cd2c96d9d
DIFF: https://github.com/llvm/llvm-project/commit/fd1ff5ee0388af9eb373d903a1644e6cd2c96d9d.diff

LOG: [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

Added: 
    

Modified: 
    llvm/lib/Transforms/IPO/Inliner.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Transforms/IPO/Inliner.cpp b/llvm/lib/Transforms/IPO/Inliner.cpp
index caba83b15111..1c9800f69e6a 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();
 }