Commit 52d871844c643f396a2bddee0753d24ff71ca3bb | zig

author	Min-Yih Hsu <minyihh@uci.edu> 2021-04-09 13:00:46-07:00
committer	Min-Yih Hsu <minyihh@uci.edu> 2021-04-09 16:51:35-07:00
log	52d871844c643f396a2bddee0753d24ff71ca3bb
tree	05adee310cd34a85fd6ced7777d669e7442f6817
parent	6b3eaa62e86d2652cf0b0802d704ab16d148c2ce

llvm new-pm: Build O0 pipeline in the correct way

Use `PassBuilder::buildO0DefaultPipeline` to build pipeline for -O0
in replacement of `PassBuilder::buildPerModuleDefaultPipeline`. This
affects both normal and LTO settings.

Two redundant Passes - which were added by accident - were also removed
from LTO pipeline.

1 files changed, 5 insertions(+), 12 deletions(-)

src
- zig_llvm.cpp

src/zig_llvm.cpp+5-12

 @@ -277,15 +277,8 @@ bool ZigLLVMTargetMachineEmitToFile(LLVMTargetMachineRef targ_machine_ref, LLVMM
         });
     }
-    // Passes for either debug or release build
+    // Passes specific for release build
-    if (is_debug) {
+    if (!is_debug) {
-      // NOTE: Always inliner will go away (in debug build)
-      // when the self-hosted compiler becomes mature.
-      pass_builder.registerPipelineStartEPCallback(
-        [](ModulePassManager &module_pm, OptimizationLevel OL) {
-          module_pm.addPass(AlwaysInlinerPass());
-        });
-    } else {
       pass_builder.registerPipelineStartEPCallback(
         [](ModulePassManager &module_pm, OptimizationLevel OL) {
           module_pm.addPass(
 @@ -312,10 +305,10 @@ bool ZigLLVMTargetMachineEmitToFile(LLVMTargetMachineRef targ_machine_ref, LLVMM
       opt_level = OptimizationLevel::O3;
     // Initialize the PassManager
-    if (lto) {
+    if (opt_level == OptimizationLevel::O0) {
+      module_pm = pass_builder.buildO0DefaultPipeline(opt_level, lto);
+    } else if (lto) {
       module_pm = pass_builder.buildLTOPreLinkDefaultPipeline(opt_level);
-      module_pm.addPass(CanonicalizeAliasesPass());
-      module_pm.addPass(NameAnonGlobalPass());
     } else {
       module_pm = pass_builder.buildPerModuleDefaultPipeline(opt_level);
     }