Skip to content

Commit eb36a73

Browse files
paperchalicechencha3
authored andcommitted
[Passes] Expose parseSinglePassOption (llvm#86117)
BPF and some machine function passes need it.
1 parent 58c8724 commit eb36a73

File tree

2 files changed

+38
-24
lines changed

2 files changed

+38
-24
lines changed

llvm/include/llvm/Passes/PassBuilder.h

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -672,6 +672,13 @@ class PassBuilder {
672672
return Result;
673673
}
674674

675+
/// Handle passes only accept one bool-valued parameter.
676+
///
677+
/// \return false when Params is empty.
678+
static Expected<bool> parseSinglePassOption(StringRef Params,
679+
StringRef OptionName,
680+
StringRef PassName);
681+
675682
private:
676683
// O1 pass pipeline
677684
FunctionPassManager

llvm/lib/Passes/PassBuilder.cpp

Lines changed: 31 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -600,8 +600,9 @@ Expected<LoopUnrollOptions> parseLoopUnrollOptions(StringRef Params) {
600600
return UnrollOpts;
601601
}
602602

603-
Expected<bool> parseSinglePassOption(StringRef Params, StringRef OptionName,
604-
StringRef PassName) {
603+
Expected<bool> PassBuilder::parseSinglePassOption(StringRef Params,
604+
StringRef OptionName,
605+
StringRef PassName) {
605606
bool Result = false;
606607
while (!Params.empty()) {
607608
StringRef ParamName;
@@ -620,24 +621,28 @@ Expected<bool> parseSinglePassOption(StringRef Params, StringRef OptionName,
620621
}
621622

622623
Expected<bool> parseGlobalDCEPassOptions(StringRef Params) {
623-
return parseSinglePassOption(Params, "vfe-linkage-unit-visibility", "GlobalDCE");
624+
return PassBuilder::parseSinglePassOption(
625+
Params, "vfe-linkage-unit-visibility", "GlobalDCE");
624626
}
625627

626628
Expected<bool> parseCGProfilePassOptions(StringRef Params) {
627-
return parseSinglePassOption(Params, "in-lto-post-link", "CGProfile");
629+
return PassBuilder::parseSinglePassOption(Params, "in-lto-post-link",
630+
"CGProfile");
628631
}
629632

630633
Expected<bool> parseInlinerPassOptions(StringRef Params) {
631-
return parseSinglePassOption(Params, "only-mandatory", "InlinerPass");
634+
return PassBuilder::parseSinglePassOption(Params, "only-mandatory",
635+
"InlinerPass");
632636
}
633637

634638
Expected<bool> parseCoroSplitPassOptions(StringRef Params) {
635-
return parseSinglePassOption(Params, "reuse-storage", "CoroSplitPass");
639+
return PassBuilder::parseSinglePassOption(Params, "reuse-storage",
640+
"CoroSplitPass");
636641
}
637642

638643
Expected<bool> parsePostOrderFunctionAttrsPassOptions(StringRef Params) {
639-
return parseSinglePassOption(Params, "skip-non-recursive-function-attrs",
640-
"PostOrderFunctionAttrs");
644+
return PassBuilder::parseSinglePassOption(
645+
Params, "skip-non-recursive-function-attrs", "PostOrderFunctionAttrs");
641646
}
642647

643648
Expected<CFGuardPass::Mechanism> parseCFGuardPassOptions(StringRef Params) {
@@ -661,19 +666,21 @@ Expected<CFGuardPass::Mechanism> parseCFGuardPassOptions(StringRef Params) {
661666
}
662667

663668
Expected<bool> parseEarlyCSEPassOptions(StringRef Params) {
664-
return parseSinglePassOption(Params, "memssa", "EarlyCSE");
669+
return PassBuilder::parseSinglePassOption(Params, "memssa", "EarlyCSE");
665670
}
666671

667672
Expected<bool> parseEntryExitInstrumenterPassOptions(StringRef Params) {
668-
return parseSinglePassOption(Params, "post-inline", "EntryExitInstrumenter");
673+
return PassBuilder::parseSinglePassOption(Params, "post-inline",
674+
"EntryExitInstrumenter");
669675
}
670676

671677
Expected<bool> parseLoopExtractorPassOptions(StringRef Params) {
672-
return parseSinglePassOption(Params, "single", "LoopExtractor");
678+
return PassBuilder::parseSinglePassOption(Params, "single", "LoopExtractor");
673679
}
674680

675681
Expected<bool> parseLowerMatrixIntrinsicsPassOptions(StringRef Params) {
676-
return parseSinglePassOption(Params, "minimal", "LowerMatrixIntrinsics");
682+
return PassBuilder::parseSinglePassOption(Params, "minimal",
683+
"LowerMatrixIntrinsics");
677684
}
678685

679686
Expected<AddressSanitizerOptions> parseASanPassOptions(StringRef Params) {
@@ -1013,13 +1020,13 @@ parseStackLifetimeOptions(StringRef Params) {
10131020
}
10141021

10151022
Expected<bool> parseDependenceAnalysisPrinterOptions(StringRef Params) {
1016-
return parseSinglePassOption(Params, "normalized-results",
1017-
"DependenceAnalysisPrinter");
1023+
return PassBuilder::parseSinglePassOption(Params, "normalized-results",
1024+
"DependenceAnalysisPrinter");
10181025
}
10191026

10201027
Expected<bool> parseSeparateConstOffsetFromGEPPassOptions(StringRef Params) {
1021-
return parseSinglePassOption(Params, "lower-gep",
1022-
"SeparateConstOffsetFromGEP");
1028+
return PassBuilder::parseSinglePassOption(Params, "lower-gep",
1029+
"SeparateConstOffsetFromGEP");
10231030
}
10241031

10251032
Expected<OptimizationLevel>
@@ -1035,13 +1042,13 @@ parseFunctionSimplificationPipelineOptions(StringRef Params) {
10351042
}
10361043

10371044
Expected<bool> parseMemorySSAPrinterPassOptions(StringRef Params) {
1038-
return parseSinglePassOption(Params, "no-ensure-optimized-uses",
1039-
"MemorySSAPrinterPass");
1045+
return PassBuilder::parseSinglePassOption(Params, "no-ensure-optimized-uses",
1046+
"MemorySSAPrinterPass");
10401047
}
10411048

10421049
Expected<bool> parseSpeculativeExecutionPassOptions(StringRef Params) {
1043-
return parseSinglePassOption(Params, "only-if-divergent-target",
1044-
"SpeculativeExecutionPass");
1050+
return PassBuilder::parseSinglePassOption(Params, "only-if-divergent-target",
1051+
"SpeculativeExecutionPass");
10451052
}
10461053

10471054
Expected<std::string> parseMemProfUsePassOptions(StringRef Params) {
@@ -1062,13 +1069,13 @@ Expected<std::string> parseMemProfUsePassOptions(StringRef Params) {
10621069
}
10631070

10641071
Expected<bool> parseStructuralHashPrinterPassOptions(StringRef Params) {
1065-
return parseSinglePassOption(Params, "detailed",
1066-
"StructuralHashPrinterPass");
1072+
return PassBuilder::parseSinglePassOption(Params, "detailed",
1073+
"StructuralHashPrinterPass");
10671074
}
10681075

10691076
Expected<bool> parseWinEHPrepareOptions(StringRef Params) {
1070-
return parseSinglePassOption(Params, "demote-catchswitch-only",
1071-
"WinEHPreparePass");
1077+
return PassBuilder::parseSinglePassOption(Params, "demote-catchswitch-only",
1078+
"WinEHPreparePass");
10721079
}
10731080

10741081
Expected<GlobalMergeOptions> parseGlobalMergeOptions(StringRef Params) {

0 commit comments

Comments
 (0)