From e7e4c7632075fef21bc8f90ad76dc68680e3bac8 Mon Sep 17 00:00:00 2001 From: Nikita Popov Date: Fri, 21 Apr 2023 10:17:03 +0200 Subject: [Pipelines] Don't run ForceFunctionAttrs post-link This is effectively a debugging pass to adjust function attributes. I don't think it makes sense to run it in the post-link pipeline. Differential Revision: https://reviews.llvm.org/D148904 --- llvm/lib/Passes/PassBuilderPipelines.cpp | 6 ------ 1 file changed, 6 deletions(-) (limited to 'llvm/lib/Passes/PassBuilderPipelines.cpp') diff --git a/llvm/lib/Passes/PassBuilderPipelines.cpp b/llvm/lib/Passes/PassBuilderPipelines.cpp index 4fa7326..4407ad0 100644 --- a/llvm/lib/Passes/PassBuilderPipelines.cpp +++ b/llvm/lib/Passes/PassBuilderPipelines.cpp @@ -1557,9 +1557,6 @@ ModulePassManager PassBuilder::buildThinLTODefaultPipeline( return MPM; } - // Force any function attributes we want the rest of the pipeline to observe. - MPM.addPass(ForceFunctionAttrsPass()); - // Add the core simplification pipeline. MPM.addPass(buildModuleSimplificationPipeline( Level, ThinOrFullLTOPhase::ThinLTOPostLink)); @@ -1626,9 +1623,6 @@ PassBuilder::buildLTODefaultPipeline(OptimizationLevel Level, // whole-program devirtualization and bitset lowering. MPM.addPass(GlobalDCEPass()); - // Force any function attributes we want the rest of the pipeline to observe. - MPM.addPass(ForceFunctionAttrsPass()); - // Do basic inference of function attributes from known properties of system // libraries and other oracles. MPM.addPass(InferFunctionAttrsPass()); -- cgit v1.1