Skip to content

Commit

Permalink
[NFC] Fix comments in PassBuilder functions (#89513)
Browse files Browse the repository at this point in the history
The original comments mention `addPreLinkLTODefaultPipeline`, but I
could not find any functions with this name, even in
https://reviews.llvm.org/D33540
(8b3be4e) where this comment was added.
I assume they meant to refer to `buildThinLTOPreLinkDefaultPipeline` and
`buildLTOPreLinkDefaultPipeline` and so this patch uses them.
  • Loading branch information
ellishg committed Apr 22, 2024
1 parent e4f7c52 commit 2e7bd22
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions llvm/include/llvm/Passes/PassBuilder.h
Original file line number Diff line number Diff line change
Expand Up @@ -264,12 +264,12 @@ class PassBuilder {
/// the LTO run.
ModulePassManager buildThinLTOPreLinkDefaultPipeline(OptimizationLevel Level);

/// Build an ThinLTO default optimization pipeline to a pass manager.
/// Build a ThinLTO default optimization pipeline to a pass manager.
///
/// This provides a good default optimization pipeline for link-time
/// optimization and code generation. It is particularly tuned to fit well
/// when IR coming into the LTO phase was first run through \c
/// addPreLinkLTODefaultPipeline, and the two coordinate closely.
/// buildThinLTOPreLinkDefaultPipeline, and the two coordinate closely.
ModulePassManager
buildThinLTODefaultPipeline(OptimizationLevel Level,
const ModuleSummaryIndex *ImportSummary);
Expand All @@ -288,7 +288,7 @@ class PassBuilder {
/// This provides a good default optimization pipeline for link-time
/// optimization and code generation. It is particularly tuned to fit well
/// when IR coming into the LTO phase was first run through \c
/// addPreLinkLTODefaultPipeline, and the two coordinate closely.
/// buildLTOPreLinkDefaultPipeline, and the two coordinate closely.
ModulePassManager buildLTODefaultPipeline(OptimizationLevel Level,
ModuleSummaryIndex *ExportSummary);

Expand Down

0 comments on commit 2e7bd22

Please sign in to comment.