Remove LTO and ArgumentPromotion passes (#4078)

LTO was unused, and ArgumentPromotion would not have any effect on DXIL
until you use noinline, and it might decompose HLSL/DXIL objects, which
could break things.

Author: tex3d

include/llvm/LinkAllPasses.h

@@ -55,7 +55,7 @@ namespace {
       (void) llvm::createBitTrackingDCEPass();
       (void) llvm::createAliasAnalysisCounterPass();
       (void) llvm::createAliasDebugger();
-      (void) llvm::createArgumentPromotionPass();
+      // (void) llvm::createArgumentPromotionPass(); // HLSL Change - do not link
       (void) llvm::createAlignmentFromAssumptionsPass();
       (void) llvm::createBasicAliasAnalysisPass();
       (void) llvm::createLibCallAliasAnalysisPass(nullptr);

lib/CMakeLists.txt

@@ -7,7 +7,7 @@ add_subdirectory(Bitcode)
 add_subdirectory(Transforms)
 add_subdirectory(Linker)
 add_subdirectory(Analysis)
-add_subdirectory(LTO)
+# add_subdirectory(LTO) # HLSL Change
 # add_subdirectory(MC) # HLSL Change
 # add_subdirectory(Object) # HLSL Change
 add_subdirectory(Option)

lib/LLVMBuild.txt

@@ -28,7 +28,6 @@ subdirectories =
  Linker
  IR
  IRReader
- LTO
  MC
  Object
  Option
@@ -48,7 +47,7 @@ subdirectories =
  DxcBindingTable
  Miniz
 
-; HLSL Change: remove LibDriver, LineEditor, add HLSL, DxrtFallback, DXIL, DxilContainer, DxilDia, DxilPIXPasses, DxilRootSignature, DxcBindingTable
+; HLSL Change: remove LibDriver, LineEditor, add HLSL, DxrtFallback, DXIL, DxilContainer, DxilDia, DxilPIXPasses, DxilRootSignature, DxcBindingTable, LTO
 
 [component_0]
 type = Group

lib/Transforms/IPO/CMakeLists.txt

@@ -1,5 +1,5 @@
+set(HLSL_IGNORE_SOURCES ArgumentPromotion.cpp)
 add_llvm_library(LLVMipo
-  ArgumentPromotion.cpp
   BarrierNoopPass.cpp
   ConstantMerge.cpp
   DeadArgumentElimination.cpp

lib/Transforms/IPO/IPO.cpp

@@ -22,7 +22,9 @@
 using namespace llvm;
 
 void llvm::initializeIPO(PassRegistry &Registry) {
+#if 0  // HLSL Change Starts: Disable ArgPromotion
   initializeArgPromotionPass(Registry);
+#endif // HLSL Change Ends
   initializeConstantMergePass(Registry);
   initializeDAEPass(Registry);
   initializeDAHPass(Registry);
@@ -53,9 +55,11 @@ void LLVMInitializeIPO(LLVMPassRegistryRef R) {
   initializeIPO(*unwrap(R));
 }
 
+#if 0  // HLSL Change Starts: Disable ArgPromotion
 void LLVMAddArgumentPromotionPass(LLVMPassManagerRef PM) {
   unwrap(PM)->add(createArgumentPromotionPass());
 }
+#endif // HLSL Change Ends
 
 void LLVMAddConstantMergePass(LLVMPassManagerRef PM) {
   unwrap(PM)->add(createConstantMergePass());

lib/Transforms/IPO/PassManagerBuilder.cpp

@@ -443,8 +443,11 @@ void PassManagerBuilder::populateModulePassManager(
   }
   if (!DisableUnitAtATime)
     MPM.add(createFunctionAttrsPass());       // Set readonly/readnone attrs
+
+#if 0  // HLSL Change Starts: Disable ArgumentPromotion
   if (OptLevel > 2)
     MPM.add(createArgumentPromotionPass());   // Scalarize uninlined fn args
+#endif // HLSL Change Ends
 
   // Start of function pass.
   // Break up aggregate allocas, using SSAUpdater.
@@ -700,6 +703,7 @@ void PassManagerBuilder::populateModulePassManager(
   addExtensionsToPM(EP_OptimizerLast, MPM);
 }
 
+#if 0 // HLSL Change: No LTO
 void PassManagerBuilder::addLTOOptimizationPasses(legacy::PassManagerBase &PM) {
   // Provide AliasAnalysis services for optimizations.
   addInitialAliasAnalysisPasses(PM);
@@ -834,6 +838,7 @@ void PassManagerBuilder::populateLTOPassManager(legacy::PassManagerBase &PM) {
   if (VerifyOutput)
     PM.add(createVerifierPass());
 }
+#endif
 
 inline PassManagerBuilder *unwrap(LLVMPassManagerBuilderRef P) {
     return reinterpret_cast<PassManagerBuilder*>(P);
@@ -910,6 +915,7 @@ LLVMPassManagerBuilderPopulateModulePassManager(LLVMPassManagerBuilderRef PMB,
   Builder->populateModulePassManager(*MPM);
 }
 
+#if 0 // HLSL Change: No LTO
 void LLVMPassManagerBuilderPopulateLTOPassManager(LLVMPassManagerBuilderRef PMB,
                                                   LLVMPassManagerRef PM,
                                                   LLVMBool Internalize,
@@ -924,3 +930,4 @@ void LLVMPassManagerBuilderPopulateLTOPassManager(LLVMPassManagerBuilderRef PMB,
 
   Builder->populateLTOPassManager(*LPM);
 }
+#endif

tools/LLVMBuild.txt

@@ -28,7 +28,6 @@ subdirectories =
  llvm-dwarfdump
  llvm-extract
  llvm-link
- llvm-lto
  llvm-mc
  llvm-mcmarkup
  llvm-nm
@@ -46,4 +45,4 @@ type = Group
 name = Tools
 parent = $ROOT
 
-; HLSL Changes: remove bugpoint, llvm-ar, llvm-jitlistener
+; HLSL Changes: remove bugpoint, llvm-ar, llvm-jitlistener, llvm-lto

tools/clang/tools/dxcompiler/CMakeLists.txt

@@ -29,7 +29,7 @@ set(LLVM_LINK_COMPONENTS
 #  libdriver
 #  lineeditor
   linker
-  lto
+#  lto
 #  mirparser # no support for LLVM codegen
   mssupport
 #  object # no support for object files (coff, elf)

tools/clang/tools/dxrfallbackcompiler/CMakeLists.txt

@@ -27,7 +27,7 @@ set(LLVM_LINK_COMPONENTS
 #  libdriver
 #  lineeditor
   linker
-  lto
+#  lto
 #  mirparser # no support for LLVM codegen
   mssupport
 #  object # no support for object files (coff, elf)

tools/clang/unittests/DxrFallback/CMakeLists.txt

@@ -15,7 +15,7 @@ set(LLVM_LINK_COMPONENTS
   ipo
   irreader
   linker
-  lto
+#  lto
   mssupport
   option
   profiledata