[llvm] 88052fc - [ArgPromo] Preserve FunctionAnalysisManagerCGSCCProxy

Arthur Eubanks via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 3 15:00:16 PDT 2021


Author: Arthur Eubanks
Date: 2021-11-03T14:56:58-07:00
New Revision: 88052fc3624a205f0e2a10d69b3ff3f80c2f7125

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

LOG: [ArgPromo] Preserve FunctionAnalysisManagerCGSCCProxy

We already make sure to properly clear analyses for deleted functions.

This makes investigating some future potential compile time improvements easier.

Reviewed By: asbirlea

Differential Revision: https://reviews.llvm.org/D113032

Added: 
    

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

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Transforms/IPO/ArgumentPromotion.cpp b/llvm/lib/Transforms/IPO/ArgumentPromotion.cpp
index dd31073d1343..0d1062e6b94b 100644
--- a/llvm/lib/Transforms/IPO/ArgumentPromotion.cpp
+++ b/llvm/lib/Transforms/IPO/ArgumentPromotion.cpp
@@ -1051,7 +1051,10 @@ PreservedAnalyses ArgumentPromotionPass::run(LazyCallGraph::SCC &C,
   if (!Changed)
     return PreservedAnalyses::all();
 
-  return PreservedAnalyses::none();
+  PreservedAnalyses PA;
+  // We've cleared out analyses for deleted functions.
+  PA.preserve<FunctionAnalysisManagerCGSCCProxy>();
+  return PA;
 }
 
 namespace {


        


More information about the llvm-commits mailing list