[llvm] [NFC][VP] Reduce parameters in `LoopVectorizePass::runImpl` (PR #103551)
via llvm-commits
llvm-commits at lists.llvm.org
Tue Aug 13 21:00:46 PDT 2024
github-actions[bot] wrote:
<!--LLVM CODE FORMAT COMMENT: {clang-format}-->
:warning: C/C++ code formatter, clang-format found issues in your code. :warning:
<details>
<summary>
You can test this locally with the following command:
</summary>
``````````bash
git-clang-format --diff 4f5d866af7fed0de1671a68530d3023e9762b71e b944d94a9ab7e3170b0b0e0cea595b1f1d72b98a --extensions cpp,h -- llvm/include/llvm/Transforms/Vectorize/LoopVectorize.h llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
``````````
</details>
<details>
<summary>
View the diff from clang-format here.
</summary>
``````````diff
diff --git a/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp b/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
index 83e41f04b2..d2e0c575a5 100644
--- a/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
+++ b/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
@@ -10236,33 +10236,33 @@ LoopVectorizeResult LoopVectorizePass::runImpl(Function &F) {
PreservedAnalyses LoopVectorizePass::run(Function &F,
FunctionAnalysisManager &AM) {
- LI = &AM.getResult<LoopAnalysis>(F);
- // There are no loops in the function. Return before computing other
- // expensive analyses.
- if (LI->empty())
- return PreservedAnalyses::all();
- SE = &AM.getResult<ScalarEvolutionAnalysis>(F);
- TTI = &AM.getResult<TargetIRAnalysis>(F);
- DT = &AM.getResult<DominatorTreeAnalysis>(F);
- TLI = &AM.getResult<TargetLibraryAnalysis>(F);
- AC = &AM.getResult<AssumptionAnalysis>(F);
- DB = &AM.getResult<DemandedBitsAnalysis>(F);
- ORE = &AM.getResult<OptimizationRemarkEmitterAnalysis>(F);
- LAIs = &AM.getResult<LoopAccessAnalysis>(F);
-
- auto &MAMProxy = AM.getResult<ModuleAnalysisManagerFunctionProxy>(F);
- PSI = MAMProxy.getCachedResult<ProfileSummaryAnalysis>(*F.getParent());
- BFI = nullptr;
- if (PSI && PSI->hasProfileSummary())
- BFI = &AM.getResult<BlockFrequencyAnalysis>(F);
- LoopVectorizeResult Result = runImpl(F);
- if (!Result.MadeAnyChange)
- return PreservedAnalyses::all();
- PreservedAnalyses PA;
-
- if (isAssignmentTrackingEnabled(*F.getParent())) {
- for (auto &BB : F)
- RemoveRedundantDbgInstrs(&BB);
+ LI = &AM.getResult<LoopAnalysis>(F);
+ // There are no loops in the function. Return before computing other
+ // expensive analyses.
+ if (LI->empty())
+ return PreservedAnalyses::all();
+ SE = &AM.getResult<ScalarEvolutionAnalysis>(F);
+ TTI = &AM.getResult<TargetIRAnalysis>(F);
+ DT = &AM.getResult<DominatorTreeAnalysis>(F);
+ TLI = &AM.getResult<TargetLibraryAnalysis>(F);
+ AC = &AM.getResult<AssumptionAnalysis>(F);
+ DB = &AM.getResult<DemandedBitsAnalysis>(F);
+ ORE = &AM.getResult<OptimizationRemarkEmitterAnalysis>(F);
+ LAIs = &AM.getResult<LoopAccessAnalysis>(F);
+
+ auto &MAMProxy = AM.getResult<ModuleAnalysisManagerFunctionProxy>(F);
+ PSI = MAMProxy.getCachedResult<ProfileSummaryAnalysis>(*F.getParent());
+ BFI = nullptr;
+ if (PSI && PSI->hasProfileSummary())
+ BFI = &AM.getResult<BlockFrequencyAnalysis>(F);
+ LoopVectorizeResult Result = runImpl(F);
+ if (!Result.MadeAnyChange)
+ return PreservedAnalyses::all();
+ PreservedAnalyses PA;
+
+ if (isAssignmentTrackingEnabled(*F.getParent())) {
+ for (auto &BB : F)
+ RemoveRedundantDbgInstrs(&BB);
}
PA.preserve<LoopAnalysis>();
@@ -10281,7 +10281,7 @@ PreservedAnalyses LoopVectorizePass::run(Function &F,
PA.preserveSet<CFGAnalyses>();
}
return PA;
- }
+}
void LoopVectorizePass::printPipeline(
raw_ostream &OS, function_ref<StringRef(StringRef)> MapClassName2PassName) {
``````````
</details>
https://github.com/llvm/llvm-project/pull/103551
More information about the llvm-commits
mailing list