PassManagerBuilder: Remove effectively dead 'StripDebug' option
authorDuncan P. N. Exon Smith <dexonsmith@apple.com>
Thu, 19 Mar 2015 21:37:17 +0000 (21:37 +0000)
committerDuncan P. N. Exon Smith <dexonsmith@apple.com>
Thu, 19 Mar 2015 21:37:17 +0000 (21:37 +0000)
`StripDebug` was only used by tools/opt/opt.cpp in
`AddStandardLinkPasses()`, but opt.cpp adds the same pass based on its
command-line flag before it calls `AddStandardLinkPasses()`.  Stripping
debug info twice isn't very useful.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@232765 91177308-0d34-0410-b5e6-96231b3b80d8

include/llvm/Transforms/IPO/PassManagerBuilder.h
lib/Transforms/IPO/PassManagerBuilder.cpp
tools/opt/opt.cpp

index 65f471268dc76aee3f41da5822dfe6bbce3848fd..a34b6644ec63a843d983779711b5d6aa104f1d0b 100644 (file)
@@ -120,7 +120,6 @@ public:
   bool DisableGVNLoadPRE;
   bool VerifyInput;
   bool VerifyOutput;
-  bool StripDebug;
   bool MergeFunctions;
 
 private:
index 5928eb1d9c3adb7c5c42392eae3964530a5c52a1..8c1e039484b2b16bc135ceb919dc396d092f5b71 100644 (file)
@@ -97,7 +97,6 @@ PassManagerBuilder::PassManagerBuilder() {
     DisableGVNLoadPRE = false;
     VerifyInput = false;
     VerifyOutput = false;
-    StripDebug = false;
     MergeFunctions = false;
 }
 
@@ -512,14 +511,10 @@ void PassManagerBuilder::populateLTOPassManager(legacy::PassManagerBase &PM) {
   if (LibraryInfo)
     PM.add(new TargetLibraryInfoWrapperPass(*LibraryInfo));
 
-  if (VerifyInput)
+  if (VerifyInput) {
     PM.add(createVerifierPass());
-
-  if (StripDebug)
-    PM.add(createStripSymbolsPass(true));
-
-  if (VerifyInput)
     PM.add(createDebugInfoVerifierPass());
+  }
 
   if (OptLevel != 0)
     addLTOOptimizationPasses(PM);
index ce057da6ceae5880be8230527923b37874224432..6c27c1ecbbfbb73c88f8e02fcc65ad82479cfb56 100644 (file)
@@ -234,7 +234,6 @@ static void AddOptimizationPasses(legacy::PassManagerBase &MPM,
 static void AddStandardLinkPasses(legacy::PassManagerBase &PM) {
   PassManagerBuilder Builder;
   Builder.VerifyInput = true;
-  Builder.StripDebug = StripDebug;
   if (DisableOptimizations)
     Builder.OptLevel = 0;