[llvm] [LTO] Remove Config.UseDefaultPipeline (PR #82587)

Igor Kudrin via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 21 22:15:10 PST 2024


https://github.com/igorkudrin created https://github.com/llvm/llvm-project/pull/82587

This option is not used. It was added in [D122133](https://reviews.llvm.org/D122133), 5856f30b, with the only usage in `ClangLinkerWrapper.cpp`, which was later updated in a1d57fc2, and then finally removed in [D142650](https://reviews.llvm.org/D142650), 6185246f.

>From 93daff745d0a07dd4ec68aa8b696dfee70f9bbdb Mon Sep 17 00:00:00 2001
From: Igor Kudrin <ikudrin at accesssoftek.com>
Date: Wed, 21 Feb 2024 19:50:56 -0800
Subject: [PATCH] [LTO] Remove Config.UseDefaultPipeline

This option is not used. It was added in https://reviews.llvm.org/D122133,
with the only usage in `ClangLinkerWrapper.cpp`, which was later removed
in https://reviews.llvm.org/D142650.
---
 llvm/include/llvm/LTO/Config.h | 3 ---
 llvm/lib/LTO/LTOBackend.cpp    | 2 --
 2 files changed, 5 deletions(-)

diff --git a/llvm/include/llvm/LTO/Config.h b/llvm/include/llvm/LTO/Config.h
index 6fb55f1cf1686a..482b6e55a19d35 100644
--- a/llvm/include/llvm/LTO/Config.h
+++ b/llvm/include/llvm/LTO/Config.h
@@ -60,9 +60,6 @@ struct Config {
   bool VerifyEach = false;
   bool DisableVerify = false;
 
-  /// Use the standard optimization pipeline.
-  bool UseDefaultPipeline = false;
-
   /// Flag to indicate that the optimizer should not assume builtins are present
   /// on the target.
   bool Freestanding = false;
diff --git a/llvm/lib/LTO/LTOBackend.cpp b/llvm/lib/LTO/LTOBackend.cpp
index 7b3a7590dfa743..6cfe67779b1a7d 100644
--- a/llvm/lib/LTO/LTOBackend.cpp
+++ b/llvm/lib/LTO/LTOBackend.cpp
@@ -330,8 +330,6 @@ static void runNewPMPasses(const Config &Conf, Module &Mod, TargetMachine *TM,
       report_fatal_error(Twine("unable to parse pass pipeline description '") +
                          Conf.OptPipeline + "': " + toString(std::move(Err)));
     }
-  } else if (Conf.UseDefaultPipeline) {
-    MPM.addPass(PB.buildPerModuleDefaultPipeline(OL));
   } else if (IsThinLTO) {
     MPM.addPass(PB.buildThinLTODefaultPipeline(OL, ImportSummary));
   } else {



More information about the llvm-commits mailing list