[llvm] [MemProf] Tolerate missing leaf debug frames (PR #71233)

Snehasish Kumar via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 3 15:13:32 PDT 2023


================
@@ -733,47 +733,52 @@ void PassBuilder::addRequiredLTOPreLinkPasses(ModulePassManager &MPM) {
   MPM.addPass(NameAnonGlobalPass());
 }
 
-void PassBuilder::addPGOInstrPasses(ModulePassManager &MPM,
-                                    OptimizationLevel Level, bool RunProfileGen,
-                                    bool IsCS, bool AtomicCounterUpdate,
-                                    std::string ProfileFile,
-                                    std::string ProfileRemappingFile,
-                                    ThinOrFullLTOPhase LTOPhase,
-                                    IntrusiveRefCntPtr<vfs::FileSystem> FS) {
+void PassBuilder::addPreInlinerPasses(ModulePassManager &MPM,
+                                      OptimizationLevel Level,
+                                      ThinOrFullLTOPhase LTOPhase) {
   assert(Level != OptimizationLevel::O0 && "Not expecting O0 here!");
-  if (!IsCS && !DisablePreInliner) {
-    InlineParams IP;
+  if (DisablePreInliner)
----------------
snehasish wrote:

If we only selectively add the PreInliner where necessary then do we need this check (and the corresponding disable-preinline flag)? I can't find any active uses of this flag apart from tests. 

https://github.com/llvm/llvm-project/pull/71233


More information about the llvm-commits mailing list