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 c23608b8d58bdeb0134d99168e6d0335da2c8366 01fd15c864b01218d36f1acd86da8476cddf97c8 -- clang/include/clang/Format/Format.h clang/lib/Format/ContinuationIndenter.cpp clang/lib/Format/Format.cpp clang/lib/Format/TokenAnnotator.cpp clang/unittests/Format/ConfigParseTest.cpp clang/unittests/Format/FormatTest.cpp clang/unittests/Format/FormatTestSelective.cpp polly/include/polly/DependenceInfo.h polly/include/polly/LinkAllPasses.h polly/lib/Analysis/DependenceInfo.cpp polly/lib/Analysis/ScopBuilder.cpp polly/lib/Analysis/ScopDetection.cpp polly/lib/CodeGen/IslNodeBuilder.cpp polly/lib/Transform/MaximalStaticExpansion.cpp polly/lib/Transform/ScheduleOptimizer.cpp `````````` </details> <details> <summary> View the diff from clang-format here. </summary> ``````````diff diff --git a/polly/include/polly/DependenceInfo.h b/polly/include/polly/DependenceInfo.h index fb7b1740ea..7526a294c6 100644 --- a/polly/include/polly/DependenceInfo.h +++ b/polly/include/polly/DependenceInfo.h @@ -332,8 +332,8 @@ namespace llvm { void initializeDependenceInfoPass(llvm::PassRegistry &); void initializeDependenceInfoPrinterLegacyPassPass(llvm::PassRegistry &); void initializeDependenceInfoWrapperPassPass(llvm::PassRegistry &); -void -initializeDependenceInfoPrinterLegacyFunctionPassPass(llvm::PassRegistry &); +void initializeDependenceInfoPrinterLegacyFunctionPassPass( + llvm::PassRegistry &); } // namespace llvm #endif diff --git a/polly/include/polly/LinkAllPasses.h b/polly/include/polly/LinkAllPasses.h index a293b7074f..a2f8f33299 100644 --- a/polly/include/polly/LinkAllPasses.h +++ b/polly/include/polly/LinkAllPasses.h @@ -138,8 +138,8 @@ void initializeJSONImporterPrinterLegacyPassPass(llvm::PassRegistry &); void initializeDependenceInfoPass(llvm::PassRegistry &); void initializeDependenceInfoPrinterLegacyPassPass(llvm::PassRegistry &); void initializeDependenceInfoWrapperPassPass(llvm::PassRegistry &); -void -initializeDependenceInfoPrinterLegacyFunctionPassPass(llvm::PassRegistry &); +void initializeDependenceInfoPrinterLegacyFunctionPassPass( + llvm::PassRegistry &); void initializeIslAstInfoWrapperPassPass(llvm::PassRegistry &); void initializeIslAstInfoPrinterLegacyPassPass(llvm::PassRegistry &); void initializeCodeGenerationPass(llvm::PassRegistry &); diff --git a/polly/lib/Analysis/DependenceInfo.cpp b/polly/lib/Analysis/DependenceInfo.cpp index 3da6738b64..69257c6038 100644 --- a/polly/lib/Analysis/DependenceInfo.cpp +++ b/polly/lib/Analysis/DependenceInfo.cpp @@ -647,9 +647,8 @@ bool Dependences::isValidSchedule(Scop &S, isl::schedule NewSched) const { return isValidSchedule(S, NewSchedules); } -bool -Dependences::isValidSchedule(Scop &S, - const StatementToIslMapTy &NewSchedule) const { +bool Dependences::isValidSchedule( + Scop &S, const StatementToIslMapTy &NewSchedule) const { if (LegalityCheckDisabled) return true; diff --git a/polly/lib/Analysis/ScopBuilder.cpp b/polly/lib/Analysis/ScopBuilder.cpp index c14b01116e..c62cb2a85c 100644 --- a/polly/lib/Analysis/ScopBuilder.cpp +++ b/polly/lib/Analysis/ScopBuilder.cpp @@ -827,9 +827,8 @@ void ScopBuilder::buildInvariantEquivalenceClasses() { } } -bool -ScopBuilder::buildDomains(Region *R, - DenseMap<BasicBlock *, isl::set> &InvalidDomainMap) { +bool ScopBuilder::buildDomains( + Region *R, DenseMap<BasicBlock *, isl::set> &InvalidDomainMap) { bool IsOnlyNonAffineRegion = scop->isNonAffineSubRegion(R); auto *EntryBB = R->getEntry(); auto *L = IsOnlyNonAffineRegion ? nullptr : LI.getLoopFor(EntryBB); @@ -3298,9 +3297,8 @@ bool ScopBuilder::buildAliasGroups() { return true; } -bool -ScopBuilder::buildAliasGroup(AliasGroupTy &AliasGroup, - DenseSet<const ScopArrayInfo *> HasWriteAccess) { +bool ScopBuilder::buildAliasGroup( + AliasGroupTy &AliasGroup, DenseSet<const ScopArrayInfo *> HasWriteAccess) { AliasGroupTy ReadOnlyAccesses; AliasGroupTy ReadWriteAccesses; SmallPtrSet<const ScopArrayInfo *, 4> ReadWriteArrays; diff --git a/polly/lib/Analysis/ScopDetection.cpp b/polly/lib/Analysis/ScopDetection.cpp index 8af4986dc1..938d3f1496 100644 --- a/polly/lib/Analysis/ScopDetection.cpp +++ b/polly/lib/Analysis/ScopDetection.cpp @@ -978,10 +978,9 @@ bool ScopDetection::hasValidArraySizes(DetectionContext &Context, // non-affine accesses are allowed, we drop the information. In case the // information is dropped the memory accesses need to be overapproximated // when translated to a polyhedral representation. -bool -ScopDetection::computeAccessFunctions(DetectionContext &Context, - const SCEVUnknown *BasePointer, - std::shared_ptr<ArrayShape> Shape) const { +bool ScopDetection::computeAccessFunctions( + DetectionContext &Context, const SCEVUnknown *BasePointer, + std::shared_ptr<ArrayShape> Shape) const { Value *BaseValue = BasePointer->getValue(); bool BasePtrHasNonAffine = false; MapInsnToMemAcc TempMemoryAccesses; @@ -1692,8 +1691,8 @@ bool ScopDetection::hasSufficientCompute(DetectionContext &Context, return InstCount >= ProfitabilityMinPerLoopInstructions; } -bool -ScopDetection::hasPossiblyDistributableLoop(DetectionContext &Context) const { +bool ScopDetection::hasPossiblyDistributableLoop( + DetectionContext &Context) const { for (auto *BB : Context.CurRegion.blocks()) { auto *L = LI.getLoopFor(BB); if (!Context.CurRegion.contains(L)) diff --git a/polly/lib/CodeGen/IslNodeBuilder.cpp b/polly/lib/CodeGen/IslNodeBuilder.cpp index 885b5ee9a1..a226cc2a1b 100644 --- a/polly/lib/CodeGen/IslNodeBuilder.cpp +++ b/polly/lib/CodeGen/IslNodeBuilder.cpp @@ -831,9 +831,8 @@ void IslNodeBuilder::createSubstitutionsVector( isl_ast_expr_free(Expr); } -void -IslNodeBuilder::generateCopyStmt(ScopStmt *Stmt, - __isl_keep isl_id_to_ast_expr *NewAccesses) { +void IslNodeBuilder::generateCopyStmt( + ScopStmt *Stmt, __isl_keep isl_id_to_ast_expr *NewAccesses) { assert(Stmt->size() == 2); auto ReadAccess = Stmt->begin(); auto WriteAccess = ReadAccess++; @@ -1130,8 +1129,8 @@ Value *IslNodeBuilder::preloadInvariantLoad(const MemoryAccess &MA, return PreloadVal; } -bool -IslNodeBuilder::preloadInvariantEquivClass(InvariantEquivClassTy &IAClass) { +bool IslNodeBuilder::preloadInvariantEquivClass( + InvariantEquivClassTy &IAClass) { // For an equivalence class of invariant loads we pre-load the representing // element with the unified execution context. However, we have to map all // elements of the class to the one preloaded load as they are referenced diff --git a/polly/lib/Transform/MaximalStaticExpansion.cpp b/polly/lib/Transform/MaximalStaticExpansion.cpp index de9b9bbc2c..e32a69d47f 100644 --- a/polly/lib/Transform/MaximalStaticExpansion.cpp +++ b/polly/lib/Transform/MaximalStaticExpansion.cpp @@ -535,8 +535,8 @@ void MaximalStaticExpanderWrapperPass::printScop(raw_ostream &OS, S.print(OS, false); } -void -MaximalStaticExpanderWrapperPass::getAnalysisUsage(AnalysisUsage &AU) const { +void MaximalStaticExpanderWrapperPass::getAnalysisUsage( + AnalysisUsage &AU) const { ScopPass::getAnalysisUsage(AU); AU.addRequired<DependenceInfo>(); AU.addRequired<OptimizationRemarkEmitterWrapperPass>(); diff --git a/polly/lib/Transform/ScheduleOptimizer.cpp b/polly/lib/Transform/ScheduleOptimizer.cpp index 751b073b7f..8ee2b66339 100644 --- a/polly/lib/Transform/ScheduleOptimizer.cpp +++ b/polly/lib/Transform/ScheduleOptimizer.cpp @@ -990,8 +990,8 @@ void IslScheduleOptimizerWrapperPass::printScop(raw_ostream &OS, Scop &) const { runScheduleOptimizerPrinter(OS, LastSchedule); } -void -IslScheduleOptimizerWrapperPass::getAnalysisUsage(AnalysisUsage &AU) const { +void IslScheduleOptimizerWrapperPass::getAnalysisUsage( + AnalysisUsage &AU) const { ScopPass::getAnalysisUsage(AU); AU.addRequired<DependenceInfo>(); AU.addRequired<TargetTransformInfoWrapperPass>(); `````````` </details> https://github.com/llvm/llvm-project/pull/78011 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits